Details

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

      Description

      Originally the course info block contained icons and "manager" with coursemanage role.

      Steps:
      1/ rename coursemanager to coursecontact role to make sure it is not being mistaken for the new manager archetype
      2/ readd some icons for each enabled enrol plugin, not sure how to deal with cost there

      Maybe we could have basic (fast for category view) and full info for one course details view.

        Gliffy Diagrams

          Attachments

          1. 201009010_MDL-22795_2.0.patch
            3 kB
          2. 20100917_MDL-22795_2.0.patch
            4 kB
          3. guestwithoutpassword.gif
            guestwithoutpassword.gif
            0.1 kB
          4. guestwithpassword.gif
            guestwithpassword.gif
            0.3 kB
          5. paidcourseaccess.gif
            paidcourseaccess.gif
            0.4 kB
          6. selfenrolwithkey.gif
            selfenrolwithkey.gif
            0.5 kB
          7. selfenrolwithkey.gif
            selfenrolwithkey.gif
            0.6 kB
          8. selfenrolwithoutkey.gif
            selfenrolwithoutkey.gif
            0.6 kB

            Activity

            Hide
            skodak Petr Skoda added a comment -

            I am no icon artist, reassigning...

            Show
            skodak Petr Skoda added a comment - I am no icon artist, reassigning...
            Hide
            skodak Petr Skoda added a comment -

            Nobody wants this, going to work on it myself then...

            Show
            skodak Petr Skoda added a comment - Nobody wants this, going to work on it myself then...
            Hide
            skodak Petr Skoda added a comment -

            The basic support is in CVS, I have to create some new icons now.

            Show
            skodak Petr Skoda added a comment - The basic support is in CVS, I have to create some new icons now.
            Hide
            dougiamas Martin Dougiamas added a comment -

            Can you describe in more detail what the icons should mean?

            Show
            dougiamas Martin Dougiamas added a comment - Can you describe in more detail what the icons should mean?
            Hide
            skodak Petr Skoda added a comment -

            I suppose we need icons only for plugins that allow some kind of "self enrolment"

            enrol/guest:

            • guest access with password
              * guest access without password

            enrol/self

            • self enrolment with key
            • self enrolment without key

            enrol/paypal
            * paid course access

            These icons should be stored in pix subdirectory in these plugins

            Show
            skodak Petr Skoda added a comment - I suppose we need icons only for plugins that allow some kind of "self enrolment" enrol/guest: guest access with password * guest access without password enrol/self self enrolment with key self enrolment without key enrol/paypal * paid course access These icons should be stored in pix subdirectory in these plugins
            Hide
            dougiamas Martin Dougiamas added a comment -

            Do we have doors and keys in http://tango.freedesktop.org/Tango_Icon_Library ?

            Show
            dougiamas Martin Dougiamas added a comment - Do we have doors and keys in http://tango.freedesktop.org/Tango_Icon_Library ?
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Hi Petr,

            I created the new icons as you requested. Please take a look and let me know if you want it differently.

            Rosie

            Show
            rwijaya Rossiani Wijaya added a comment - Hi Petr, I created the new icons as you requested. Please take a look and let me know if you want it differently. Rosie
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Attached patch to display the enrolment icons.

            Create 'pix' directory for the following plugins for the above icons:

            • /enrol/guest/pix/guest.gif
            • /enrol/guest/pix/guestwithpassword.gif
            • /enrol/self/pix/self.gif
            • /enrol/self/pix/selfwithkey.gif
            • /enrol/paypal/pix/payaccess.gif

            Please let me know if it needs any changes.

            Rosie

            Show
            rwijaya Rossiani Wijaya added a comment - Attached patch to display the enrolment icons. Create 'pix' directory for the following plugins for the above icons: /enrol/guest/pix/guest.gif /enrol/guest/pix/guestwithpassword.gif /enrol/self/pix/self.gif /enrol/self/pix/selfwithkey.gif /enrol/paypal/pix/payaccess.gif Please let me know if it needs any changes. Rosie
            Hide
            dougiamas Martin Dougiamas added a comment -

            Petr can you check this out?

            Show
            dougiamas Martin Dougiamas added a comment - Petr can you check this out?
            Hide
            dougiamas Martin Dougiamas added a comment -

            selfenrolwithkey.gif could be just a key by itself.

            Show
            dougiamas Martin Dougiamas added a comment - selfenrolwithkey.gif could be just a key by itself.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Attach new 'selfenrolwithkey" icon.

            Show
            rwijaya Rossiani Wijaya added a comment - Attach new 'selfenrolwithkey" icon.
            Hide
            skodak Petr Skoda added a comment -

            ok, I will patch the code to use these icons

            Show
            skodak Petr Skoda added a comment - ok, I will patch the code to use these icons
            Hide
            dougiamas Martin Dougiamas added a comment -

            Can you finish this, Petr? Or should Rosie do it?

            Show
            dougiamas Martin Dougiamas added a comment - Can you finish this, Petr? Or should Rosie do it?
            Hide
            skodak Petr Skoda added a comment - - edited

            Ok, so the icons are in CVS, the plugins are returning them, now we need to make sure they are printed where necessary, thi list of places is:
            1/ frontpage - list of courses for not logged in user and admin
            2/ frontpage - combo list for not logged in user and admin
            3/ course category view (course/category.php) - for all users

            I suppose it does not make much sense to print these icons next to "My courses" list.

            The actual code that gives you the icons is

            if ($icons = enrol_get_course_info_icons($acourse)) {
              foreach ($icons as $pix_icon) {
                 echo $OUTPUT->render($pix_icon);
              }
            }

            It should be in some places already, but the CSS is probably incorrect there and I might have misplaced the code a bit too...

            Show
            skodak Petr Skoda added a comment - - edited Ok, so the icons are in CVS, the plugins are returning them, now we need to make sure they are printed where necessary, thi list of places is: 1/ frontpage - list of courses for not logged in user and admin 2/ frontpage - combo list for not logged in user and admin 3/ course category view (course/category.php) - for all users I suppose it does not make much sense to print these icons next to "My courses" list. The actual code that gives you the icons is if ($icons = enrol_get_course_info_icons($acourse)) { foreach ($icons as $pix_icon) { echo $OUTPUT->render($pix_icon); } } It should be in some places already, but the CSS is probably incorrect there and I might have misplaced the code a bit too...
            Hide
            skodak Petr Skoda added a comment -

            Sam: could you please review patches and send to cvs if I am not online? It would be nice to get these fixes in RC, ciao and thanks everybody.

            Show
            skodak Petr Skoda added a comment - Sam: could you please review patches and send to cvs if I am not online? It would be nice to get these fixes in RC, ciao and thanks everybody.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Update patch, as suggested by Petr.

            Hi Petr/Sam,
            The only icon that hasn't been set is 'paid course icon'. I'm not too sure how to set a paid course in icon. Could you provide some info regarding this course? Once I understand how it works, I could implement the patch for the paid course.

            Thanks.
            Rosie

            Show
            rwijaya Rossiani Wijaya added a comment - Update patch, as suggested by Petr. Hi Petr/Sam, The only icon that hasn't been set is 'paid course icon'. I'm not too sure how to set a paid course in icon. Could you provide some info regarding this course? Once I understand how it works, I could implement the patch for the paid course. Thanks. Rosie
            Hide
            skodak Petr Skoda added a comment -

            paid course is mostly paypal, we might just use paypal icon
            each payment plugin might use different icon - or we might just use one icon in course lists and service specific icons on the enrolment page...

            Show
            skodak Petr Skoda added a comment - paid course is mostly paypal, we might just use paypal icon each payment plugin might use different icon - or we might just use one icon in course lists and service specific icons on the enrolment page...
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Hi guys,

            Looks like you two were working on the same thing at the same time, Rosie's patch is doing what Petr's commits are doing.

            I've just commit a patch that adds the icons for the available courses (agree my courses makes no sense).
            Having read this issue it looks at though everything is done as described, I'll leave it open presently but perhaps Petr/Rosie you could double check everything has been done.

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Hi guys, Looks like you two were working on the same thing at the same time, Rosie's patch is doing what Petr's commits are doing. I've just commit a patch that adds the icons for the available courses (agree my courses makes no sense). Having read this issue it looks at though everything is done as described, I'll leave it open presently but perhaps Petr/Rosie you could double check everything has been done. Cheers Sam
            Hide
            rwijaya Rossiani Wijaya added a comment -

            All necessary code fix and icons has been added since pre-release of 2.0.

            Tested and it works.

            Thanks everyone

            Rosie

            resolved.

            Show
            rwijaya Rossiani Wijaya added a comment - All necessary code fix and icons has been added since pre-release of 2.0. Tested and it works. Thanks everyone Rosie resolved.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10