Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Calendar
    • Labels:
    • Testing Instructions:
      Hide
      1. Login as admin
      2. Go to Navigation > Site pages > Calendar.
      3. Click the button 'Manage subscriptions'.
      4. Try importing a calendar with the URL http://www.mozilla.org/projects/calendar/caldata/AustraliaHolidays.ics , set "Poll interval" to weekly
      5. Now change "Poll interval" for this subscription to hourly and click update
      6. Make sure this "Poll interval" is set to hourly.
      Show
      Login as admin Go to Navigation > Site pages > Calendar. Click the button 'Manage subscriptions'. Try importing a calendar with the URL http://www.mozilla.org/projects/calendar/caldata/AustraliaHolidays.ics , set "Poll interval" to weekly Now change "Poll interval" for this subscription to hourly and click update Make sure this "Poll interval" is set to hourly.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      wip-mdl-36351

      Description

      As commented by Raj in MDL-16660:

      Poll interval can't be updated. Clicking update sets poll interval to never (0)

      1. Add ical subscription with "Poll interval" to hourly
      2. Change poll to weekly, and click update. Poll is set to never.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              ankit_frenz Ankit Agarwal added a comment -

              Hi Raj,
              This is looking good.
              Please feel free to submit for integration.
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - Hi Raj, This is looking good. Please feel free to submit for integration. Thanks
              Hide
              rajeshtaneja Rajesh Taneja added a comment -

              Thanks Ankit, pushing for integration

              Show
              rajeshtaneja Rajesh Taneja added a comment - Thanks Ankit, pushing for integration
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks Raj, this has been integrated now

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks Raj, this has been integrated now
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Passing tests as the QA issue will cover it.

              Show
              samhemelryk Sam Hemelryk added a comment - Passing tests as the QA issue will cover it.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              (also noting I tested it anyway )

              Show
              samhemelryk Sam Hemelryk added a comment - (also noting I tested it anyway )
              Hide
              poltawski Dan Poltawski added a comment -

              Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week!

              ciao
              Dan

              Show
              poltawski Dan Poltawski added a comment - Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week! ciao Dan

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    3/Dec/12