Moodle
  1. Moodle
  2. MDL-39581

It's possible for the calendar popup to change multiple date elements.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Forms Library
    • Labels:
      None
    • Testing Instructions:
      Hide

      There is no real way to test this because the behaviour of the pop-up has changed. Previously when you had multiple date elements enabled on a form, and clicked on a calendar image so the pop-up appeared you could then click on another element's calendar image and the first element's pop-up would disappear but another would not appear until you clicked the image again. Meaning you had to click twice for the date pop-up to appear for the second date element you selected. Now, when you click on any calendar image, any previously opened date selectors for other elements that were open will be closed and a new pop-up will appear.

      Test 1

      1. Go to Site administration ► Users ► Accounts ► Browse list of users.
      2. Click "Show more"
      3. Enable all the date fields.
      4. Change the dates and ensure the date only changes for the one you selected. Alternate between changing dates in the pop-up and the select boxes.

      Test 2

      1. Create a feedback activity.
      2. On the settings page enable all date fields.
      3. Change the dates and ensure the date only changes for the one you selected. Alternate between changing dates in the pop-up and the select boxes.

      Test 3

      1. Repeat the steps in MDL-26649

      Please test in at least ie, firefox and chrome.

      Show
      There is no real way to test this because the behaviour of the pop-up has changed. Previously when you had multiple date elements enabled on a form, and clicked on a calendar image so the pop-up appeared you could then click on another element's calendar image and the first element's pop-up would disappear but another would not appear until you clicked the image again. Meaning you had to click twice for the date pop-up to appear for the second date element you selected. Now, when you click on any calendar image, any previously opened date selectors for other elements that were open will be closed and a new pop-up will appear. Test 1 Go to Site administration ► Users ► Accounts ► Browse list of users. Click "Show more" Enable all the date fields. Change the dates and ensure the date only changes for the one you selected. Alternate between changing dates in the pop-up and the select boxes. Test 2 Create a feedback activity. On the settings page enable all date fields. Change the dates and ensure the date only changes for the one you selected. Alternate between changing dates in the pop-up and the select boxes. Test 3 Repeat the steps in MDL-26649 Please test in at least ie, firefox and chrome.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-39581_master
    • Rank:
      50267

      Description

      1. Go to Site administration ► Users ► Accounts ► Browse list of users
      2. Click "Show more"
      3. Enable all the date fields.
      4. Click on calender icon for the lowest date selector.
      5. Click on the icon for the one above that so that the calendar closes.
      6. Click on the icon again so the calendar pop ups.
      7. Repeat the last 2 steps until you have the calendar open on the top date selector.
      8. Change the date and notice all the select boxes change.

        Issue Links

          Activity

          Hide
          Rajesh Taneja added a comment -

          Thanks Mark Patch looks good to me.
          Pushing it for integration.

          Show
          Rajesh Taneja added a comment - Thanks Mark Patch looks good to me. Pushing it for integration.
          Hide
          Eloy Lafuente (stronk7) 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
          Eloy Lafuente (stronk7) 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
          Damyon Wiese added a comment -

          Thanks Mark,

          This has been integrated to master. I the user search page in chrome and the fix worked well.

          Show
          Damyon Wiese added a comment - Thanks Mark, This has been integrated to master. I the user search page in chrome and the fix worked well.
          Hide
          David Monllaó added a comment -

          It passes using IE9

          Show
          David Monllaó added a comment - It passes using IE9
          Hide
          David Monllaó added a comment -

          Using IE9 in Win7, the only thing I've noticed is that with the date selector enabled and the popup calendar opened, if I uncheck 'Enable' the popup calendar is still opened and I need an extra click to close it; if after unchecking 'Enable' I select a date using the popup calendar the date is updated in the disabled (disabled since I unchecked 'Enable') drop down menus.

          Show
          David Monllaó added a comment - Using IE9 in Win7, the only thing I've noticed is that with the date selector enabled and the popup calendar opened, if I uncheck 'Enable' the popup calendar is still opened and I need an extra click to close it; if after unchecking 'Enable' I select a date using the popup calendar the date is updated in the disabled (disabled since I unchecked 'Enable') drop down menus.
          Hide
          Adrian Greeve added a comment -

          Tested on firefox and Chrome, I get the same behaviour as David mentions, but I think that can be tackled in a separate issue if required.
          Test passed.

          Show
          Adrian Greeve added a comment - Tested on firefox and Chrome, I get the same behaviour as David mentions, but I think that can be tackled in a separate issue if required. Test passed.
          Hide
          Damyon Wiese added a comment -

          Added an new issue for the reported behaviour.

          Show
          Damyon Wiese added a comment - Added an new issue for the reported behaviour.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Just In Time® for stable releases, thanks!

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Just In Time® for stable releases, thanks! Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: