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

The english update timezones message tries to reasure but puts doubts in my mind

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.9.5, 2.2
    • Fix Version/s: 2.2.3
    • Component/s: Language
    • Labels:

      Description

      $string['configintrotimezones'] = 'This page will search for new information about world timezones (including daylight savings time rules) and update your local database with this information. These locations will be checked, in order: $a This procedure is generally very safe and can not break normal installations. Do you wish to update your timezones now?';

      I think it would reassure me more if all operations were safe and don't need a message telling users it wont break their installation! Could we change to:

      $string['configintrotimezones'] = 'This page will search for new information about world timezones (including daylight savings time rules) and update your local database with this information. These locations will be checked, in order: $a Do you wish to update your timezones now?';

      Votes, please!

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting this issue.

            We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

            If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

            Michael d;

            lqjjLKA0p6

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; lqjjLKA0p6
            Hide
            poltawski Dan Poltawski added a comment -

            Still in the lang file in 2.2

            Show
            poltawski Dan Poltawski added a comment - Still in the lang file in 2.2
            Hide
            poltawski Dan Poltawski added a comment -

            Hi Helen,

            What do you think about my suggested lanaguage change? (from ages ago!)

            Show
            poltawski Dan Poltawski added a comment - Hi Helen, What do you think about my suggested lanaguage change? (from ages ago!)
            Hide
            tsala Helen Foster added a comment -

            Hi Dan,

            Removing 'This procedure is generally very safe and can not break normal installations.' has my vote! Sorry for taking so long to reply.

            Show
            tsala Helen Foster added a comment - Hi Dan, Removing 'This procedure is generally very safe and can not break normal installations.' has my vote! Sorry for taking so long to reply.
            Hide
            poltawski Dan Poltawski added a comment -

            I've fixed this for 2.2 and master. Since things have been moved around and this string has been around for ages I didn't bother with 2.1

            Show
            poltawski Dan Poltawski added a comment - I've fixed this for 2.2 and master. Since things have been moved around and this string has been around for ages I didn't bother with 2.1
            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
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (22 & master), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (22 & master), thanks!
            Hide
            phalacee Jason Fowler added a comment -

            looks good

            Show
            phalacee Jason Fowler added a comment - looks good
            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:
                1 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

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