Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.4
    • Component/s: Messages
    • Labels:
    • Database:
      Any
    • Testing Instructions:
      Hide

      Go to messaging (My profile > Messages) and send another user a personal message.

      Log in as that user, go to My Profile > Messages and check that the message appears for them. It doesnt matter whether its read or not, just that the message appears at all.

      Show
      Go to messaging (My profile > Messages) and send another user a personal message. Log in as that user, go to My Profile > Messages and check that the message appears for them. It doesnt matter whether its read or not, just that the message appears at all.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      MDL-31033_message_deprecated

      Description

      It contains a bunch of commented out code that can probably just be deleted. Find out why its there and whether it can just be removed.

        Gliffy Diagrams

          Activity

          Hide
          Andrew Davis added a comment -

          Adding a branch and testing instructions. Putting this up for peer review.

          Show
          Andrew Davis added a comment - Adding a branch and testing instructions. Putting this up for peer review.
          Hide
          Ankit Agarwal added a comment -

          Hi Andrew,
          Do we need to overwrite the set_data and get_data methods when they are simply calling parent methods?
          will it make more sense to remove them completely?

          thanks

          Show
          Ankit Agarwal added a comment - Hi Andrew, Do we need to overwrite the set_data and get_data methods when they are simply calling parent methods? will it make more sense to remove them completely? thanks
          Hide
          Andrew Davis added a comment -

          Those overrides are rather pointless arent they. Ive removed them. Submitting for integration.

          Show
          Andrew Davis added a comment - Those overrides are rather pointless arent they. Ive removed them. Submitting for integration.
          Hide
          Sam Hemelryk added a comment -

          Hi Andrew,

          Just double checking, but the only thing happening here is that code is being cleaned up.
          If so then I may stop the review on this and leave it until after the minor releases at the end of this week.
          As master and 23 have to stay in sync it keeps things simpler if left until after then.

          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Hi Andrew, Just double checking, but the only thing happening here is that code is being cleaned up. If so then I may stop the review on this and leave it until after the minor releases at the end of this week. As master and 23 have to stay in sync it keeps things simpler if left until after then. Cheers Sam
          Hide
          Andrew Davis added a comment -

          Yep, its just clean up. No rush.

          Show
          Andrew Davis added a comment - Yep, its just clean up. No rush.
          Hide
          Sam Hemelryk added a comment -

          Cool thanks Andrew, I'll stop the review on this now and shift it back to next week (when it can go in for sure!)

          Show
          Sam Hemelryk added a comment - Cool thanks Andrew, I'll stop the review on this now and shift it back to next week (when it can go in for sure!)
          Hide
          Sam Hemelryk added a comment -

          Thanks Andrew, this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks Andrew, this has been integrated now.
          Hide
          Frédéric Massart added a comment -

          Passed. Thanks guys!

          Show
          Frédéric Massart added a comment - Passed. Thanks guys!
          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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: