Moodle
  1. Moodle
  2. MDL-27831

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial 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
    • Rank:
      17473

      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.

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

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

          Show
          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 Agarwal added a comment - - edited

          Estimated development time:- 3 hour

          Show
          Ankit Agarwal added a comment - - edited Estimated development time:- 3 hour
          Hide
          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 Agarwal added a comment - - edited @integrator This must land after MDL-27842 After which this can be cherry-picked to stables. Thanks
          Hide
          Rossiani Wijaya added a comment -

          This looks good.

          +1 for integration review.

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

          Thanks for the review Rosie.
          Submitting for integration.
          Thanks

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

          rebased.
          Thanks

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

          Thanks Ankit, integrated to master

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

          22 and 23 too (after seeing your comment)

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

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

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

          works fine

          Show
          Jason Fowler added a comment - works fine
          Hide
          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
          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 Agarwal added a comment -

          @Dan
          That was so inspiring I feel alive again!

          Show
          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: