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

Delete Event - Delete button displayed in multiple places in a page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.4
    • Component/s: Calendar
    • Labels:
    • Environment:
      Windows XP
    • Database:
      MySQL
    • Testing Instructions:
      Hide

      1. Ensure debugging is turned on at Site Administration -> Development -> Debugging
      2. Click on a ling in the calendar block to be taken to the list of events
      3. Click the button "New event" button
      4. Enter details for a new event and save
      5. Find the new event in the events list
      6. Click the X icon on the right of the event
      7. The event information should be shown without any errors reported
      8. Make sure on "are you sure" page you dont see an x icon along with event details (to delete the event)

      Show
      1. Ensure debugging is turned on at Site Administration -> Development -> Debugging 2. Click on a ling in the calendar block to be taken to the list of events 3. Click the button "New event" button 4. Enter details for a new event and save 5. Find the new event in the events list 6. Click the X icon on the right of the event 7. The event information should be shown without any errors reported 8. Make sure on "are you sure" page you dont see an x icon along with event details (to delete the event)
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      MDL-27831-master

      Description

      Steps to reproduce

      1. Click on X delete icon for an event
      2. User will be redirected to a page where it asks for "Are you sure you want to delete this event"
      3. A "Delete" button and one more X delete icon is displayed on the same page.
      4. When user clicks on the X delete icon, user will be redirected to the same page.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            The event could be displayed without the config and delete icons, or perhaps not displayed at all.

            Show
            salvetore Michael de Raadt added a comment - The event could be displayed without the config and delete icons, or perhaps not displayed at all.
            Hide
            ankit_frenz Ankit Agarwal added a comment - - edited

            Estimated development time:- 3 hour

            Show
            ankit_frenz Ankit Agarwal added a comment - - edited Estimated development time:- 3 hour
            Hide
            ankit_frenz Ankit Agarwal added a comment - - edited

            @integrator
            This must land after MDL-27842
            After which this can be cherry-picked to stables.

            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - - edited @integrator This must land after MDL-27842 After which this can be cherry-picked to stables. Thanks
            Hide
            rwijaya Rossiani Wijaya added a comment -

            This looks good.

            +1 for integration review.

            Show
            rwijaya Rossiani Wijaya added a comment - This looks good. +1 for integration review.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Thanks for the review Rosie.
            Submitting for integration.
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - Thanks for the review Rosie. Submitting for integration. Thanks
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            rebased.
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - rebased. Thanks
            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Ankit, integrated to master

            Show
            poltawski Dan Poltawski added a comment - Thanks Ankit, integrated to master
            Hide
            poltawski Dan Poltawski added a comment -

            22 and 23 too (after seeing your comment)

            Show
            poltawski Dan Poltawski added a comment - 22 and 23 too (after seeing your comment)
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            taking myself off as tester. As I cannot test my own issue.
            Thnaks

            Show
            ankit_frenz Ankit Agarwal added a comment - taking myself off as tester. As I cannot test my own issue. Thnaks
            Hide
            phalacee Jason Fowler added a comment -

            works fine

            Show
            phalacee Jason Fowler added a comment - works fine
            Hide
            poltawski Dan Poltawski added a comment -

            Congratulations!

            You've made it into the weekly release!

            Thanks for your contribution - here are some random drummers to keep you inspired for the next week!
            http://www.youtube.com/watch?v=_QhpHUmVCmY

            Show
            poltawski Dan Poltawski added a comment - Congratulations! You've made it into the weekly release! Thanks for your contribution - here are some random drummers to keep you inspired for the next week! http://www.youtube.com/watch?v=_QhpHUmVCmY
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            @Dan
            That was so inspiring I feel alive again!

            Show
            ankit_frenz Ankit Agarwal added a comment - @Dan That was so inspiring I feel alive again!

              People

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

                Dates

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