Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-36626 Meta: Ical Bug fixes and improvments
  3. MDL-36351

Poll interval in calendar subscription cannot be updated

          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 from Repository:
          • Pull Master Branch:
            wip-mdl-36351
          • Pull Master Diff URL:

            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

                  has a non-specific relationship to

                  Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-36352 Poll interval should be disabled for calendar file import

                  • Minor - Minor loss of function where workaround is possible
                  • Closed
                  Testing discovered

                  New Feature - A new Moodle feature which has yet to be developed. MDL-16660 Calendar: IMPORT or Subscribe to iCal feed into Moodle calendar

                  • Blocker - Blocks development and/or testing, prevents Moodle from running (applicable to bugs only)
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    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
                    Permalink
                    rajeshtaneja Rajesh Taneja added a comment -

                    Thanks Ankit, pushing for integration

                    Show
                    rajeshtaneja Rajesh Taneja added a comment - Thanks Ankit, pushing for integration
                    Hide
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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