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

          Attachments

            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