Details

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

      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
      1. authorize_20100817.patch
        164 kB
        Eugene Venter
      2. authorize_2010081701.patch
        164 kB
        Eugene Venter

        Activity

        Hide
        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
        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
        Petr Škoda added a comment -

        I am proposing to move this out to contrib.

        Show
        Petr Škoda added a comment - I am proposing to move this out to contrib.
        Hide
        Martin Dougiamas added a comment -

        Aaron, are you able to work on this?

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

        Hi Martin,

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

        Cheers,
        Aaron

        Show
        Aaron Barnes added a comment - Hi Martin, I can't, but Eugene should be able to. Cheers, Aaron
        Hide
        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
        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
        Eugene Venter added a comment -

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

        Show
        Eugene Venter added a comment - It looks like this should be similar to the PayPal plugin, so ignore my question above.
        Hide
        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
        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
        Eugene Venter added a comment -

        Hi, please see the latest, updated patch

        Thanks,
        Eugene.

        Show
        Eugene Venter added a comment - Hi, please see the latest, updated patch Thanks, Eugene.
        Hide
        Petr Škoda 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
        Petr Škoda 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
        Petr Škoda added a comment -

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

        Petr Skoda

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

        Hi Petr,

        I'll commit it this morning

        Cheers,
        Aaron

        Show
        Aaron Barnes added a comment - Hi Petr, I'll commit it this morning Cheers, Aaron
        Hide
        Petr Škoda added a comment -

        Thank you.

        Show
        Petr Škoda added a comment - Thank you.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: