Moodle

Clear out /message/send_form.php

Details

  • Type: Sub-task Sub-task
  • Status: Closed 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 from Repository:
  • Pull Master Branch:
    MDL-31033_message_deprecated
  • Pull Master Diff URL:

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.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
Frédéric Massart added a comment -

Passed. Thanks guys!

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

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: