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

          Attachments

            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: