Moodle
  1. Moodle
  2. MDL-16715

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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
          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
          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
          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
          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 Cai added a comment -

          Thanks all, closing

          Show
          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: