Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-32494

CAS configuration page mis-spellings (MDL-16646 reloaded)

    Details

    • Testing Instructions:
      Hide

      1. Go to "Site Administration >> Plugins >> Authentication >> Manage authenication".
      2. Click on the "CAS Server (SSO)" settings link on the right (no need to activate the plugin).
      3. Check the new spelling and strings.
      4. Profit!

      Show
      1. Go to "Site Administration >> Plugins >> Authentication >> Manage authenication". 2. Click on the "CAS Server (SSO)" settings link on the right (no need to activate the plugin). 3. Check the new spelling and strings. 4. Profit!
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip_mdl-16646_master

      Description

      As directed in MDL-16646 by Michael, I'm opening a new issue (and proposing a patch for it). Quoting from MDL-16646:

      There are several mis-spellings, grammatical errors, and confusing language on the CAS configuration page.

      1. Version - this currently says "Version of CAS". This is very confusing because what it is really asking for is the protocol version of CAS not the server version. This should say something like "CAS protocol version" The field label should be changed as well to reflect what is really being asked for.

      2.Proxy mode - Currently says "Turn this to 'yes'' if you use CASin proxy-mode" should be something like "Select "Yes" if you use CAS in proxy mode."

      3. Logout CAS - Currently says "Turn this to 'yes'' if tou want to logout from CAS when you deconnect from Moodle" should say something like "Select "Yes" if you want to logout of CAS when you disconnect from Moodle" The field label should be changed to "CAS Logout Option"

      4. Multi-Authentication - Currently says "Turn this to 'yes'' if you want to have multi-authentication (CAS + other authentication)" should say something like "Select "Yes" if you use multiple authentication plug-ins (CAS + other authentication)"

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for persisting with that and providing a patch.

            Show
            salvetore Michael de Raadt added a comment - Thanks for persisting with that and providing a patch.
            Hide
            tsala Helen Foster added a comment -

            Iñaki, thanks for your patch. The strings all look great to me, apart from 'CAS Protocol Version' which I think should be 'CAS protocol version' according to http://docs.moodle.org/dev/Coding_style#Language_strings

            Show
            tsala Helen Foster added a comment - Iñaki, thanks for your patch. The strings all look great to me, apart from 'CAS Protocol Version' which I think should be 'CAS protocol version' according to http://docs.moodle.org/dev/Coding_style#Language_strings
            Hide
            iarenaza Iñaki Arenaza added a comment -

            Thanks Helen

            I'll change the string and will resubmit the changes to github today.

            Show
            iarenaza Iñaki Arenaza added a comment - Thanks Helen I'll change the string and will resubmit the changes to github today.
            Hide
            iarenaza Iñaki Arenaza added a comment -

            I have just pushed the changes to github (same branch as before).

            Saludos.
            Iñaki.

            Show
            iarenaza Iñaki Arenaza added a comment - I have just pushed the changes to github (same branch as before). Saludos. Iñaki.
            Hide
            poltawski Dan Poltawski added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Iñaki, i've integrated this now - including to 2.1 and 2.2

            Show
            poltawski Dan Poltawski added a comment - Thanks Iñaki, i've integrated this now - including to 2.1 and 2.2
            Hide
            rwijaya Rossiani Wijaya added a comment -

            This looks good.

            Thanks Inaki.

            Test passed.

            Show
            rwijaya Rossiani Wijaya added a comment - This looks good. Thanks Inaki. Test passed.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This has been near becoming rejected, because it's not the best code you are able to produce.

            But, luckily, at the end, it has landed and has been spread to all repos out there.

            Many thanks and, don't forget it, keep improving your skills, you can!

            Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This has been near becoming rejected, because it's not the best code you are able to produce. But, luckily, at the end, it has landed and has been spread to all repos out there. Many thanks and, don't forget it, keep improving your skills, you can! Closing, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12