Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-16715

Stating in October 2008, authorize.net codes need true 10cc integers.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.6, 1.9.2
    • Fix Version/s: 1.8.7, 1.9.3
    • Component/s: Enrolments
    • Labels:
      None
    • Environment:
      Any Moodle using authorize enrol plugin
    • Database:
      Any
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      Until now, it was enough with 2,147,483,647. Starting in October 2008 true 10cc numbers (9,999,999,999) must be supported.

      Ethem is going to change those transid fields to int(20) to support future expansions.

      References:

        Gliffy Diagrams

          Activity

          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Ethem has committed the change to 19_STABLE and HEAD. I've executed the upgrade twice under all 4 DB flavours (Mysql, PostgreSQL, MSSQL and Oracle) and worked perfectly.

          Going to backport the changes to 18_STABLE too.

          Ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Ethem has committed the change to 19_STABLE and HEAD. I've executed the upgrade twice under all 4 DB flavours (Mysql, PostgreSQL, MSSQL and Oracle) and worked perfectly. Going to backport the changes to 18_STABLE too. Ciao
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Changes have been backported to 18_STABLE too. Resolving this as fixed.

          Thanks Ethem Evlice (hope I wrote this properly this time) :-P

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Changes have been backported to 18_STABLE too. Resolving this as fixed. Thanks Ethem Evlice (hope I wrote this properly this time) :-P
          Hide
          dongsheng Dongsheng Cai added a comment -

          Thanks all, closing

          Show
          dongsheng Dongsheng Cai added a comment - Thanks all, closing

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                15/Oct/08