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

          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