Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: None
    • Component/s: Enrolments
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE

      Description

      overview:

      • enrol/authorize/* needs to be updated to use new enrol_plugin base class - the code is now in lib.php instead of enrol.php
      • old hardcoded hooks for authorize enrol plugins were removed from core, it should be possible to keep everything in enrol/authorize now

        Gliffy Diagrams

        1. authorize_20100817.patch
          164 kB
          Eugene Venter
        2. authorize_2010081701.patch
          164 kB
          Eugene Venter

          Activity

          Hide
          dougiamas Martin Dougiamas added a comment -

          I think the author of this quit some time ago. Aaron, can you look at this one too, once you become an enrolment plugins expert from the other ones?

          Show
          dougiamas Martin Dougiamas added a comment - I think the author of this quit some time ago. Aaron, can you look at this one too, once you become an enrolment plugins expert from the other ones?
          Hide
          skodak Petr Skoda added a comment -

          I am proposing to move this out to contrib.

          Show
          skodak Petr Skoda added a comment - I am proposing to move this out to contrib.
          Hide
          dougiamas Martin Dougiamas added a comment -

          Aaron, are you able to work on this?

          Show
          dougiamas Martin Dougiamas added a comment - Aaron, are you able to work on this?
          Hide
          sry_not4sale Aaron Barnes added a comment -

          Hi Martin,

          I can't, but Eugene should be able to.

          Cheers,
          Aaron

          Show
          sry_not4sale Aaron Barnes added a comment - Hi Martin, I can't, but Eugene should be able to. Cheers, Aaron
          Hide
          binare Eugene Venter added a comment -

          Hi there,

          I will try and give this one a go .

          Should this plugin have multiple instances as well?

          Thanks!
          Eugene.

          Show
          binare Eugene Venter added a comment - Hi there, I will try and give this one a go . Should this plugin have multiple instances as well? Thanks! Eugene.
          Hide
          binare Eugene Venter added a comment -

          It looks like this should be similar to the PayPal plugin, so ignore my question above.

          Show
          binare Eugene Venter added a comment - It looks like this should be similar to the PayPal plugin, so ignore my question above.
          Hide
          binare Eugene Venter added a comment -

          Hey Martin/Petr,

          Please see the attached patch for porting this plugin to 2.0.

          I have done some light testing with a Authorize.Net test account, but not extensive .

          Hope this helps !

          Cheers,
          Eugene.

          Show
          binare Eugene Venter added a comment - Hey Martin/Petr, Please see the attached patch for porting this plugin to 2.0. I have done some light testing with a Authorize.Net test account, but not extensive . Hope this helps ! Cheers, Eugene.
          Hide
          binare Eugene Venter added a comment -

          Hi, please see the latest, updated patch

          Thanks,
          Eugene.

          Show
          binare Eugene Venter added a comment - Hi, please see the latest, updated patch Thanks, Eugene.
          Hide
          skodak Petr Skoda added a comment -

          Looks ok, I did not test it because I know nothing about this payment service.
          Thanks, +1 for commit.

          Petr Skoda

          Show
          skodak Petr Skoda added a comment - Looks ok, I did not test it because I know nothing about this payment service. Thanks, +1 for commit. Petr Skoda
          Hide
          skodak Petr Skoda added a comment -

          Hello,
          what is the status of this patch, when are you planning to commit it?

          Petr Skoda

          Show
          skodak Petr Skoda added a comment - Hello, what is the status of this patch, when are you planning to commit it? Petr Skoda
          Hide
          sry_not4sale Aaron Barnes added a comment -

          Hi Petr,

          I'll commit it this morning

          Cheers,
          Aaron

          Show
          sry_not4sale Aaron Barnes added a comment - Hi Petr, I'll commit it this morning Cheers, Aaron
          Hide
          skodak Petr Skoda added a comment -

          Thank you.

          Show
          skodak Petr Skoda added a comment - Thank you.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: