Moodle

Difficulty troubleshooting sending of messages via user interface

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.6
  • Fix Version/s: None
  • Component/s: Unknown
  • Labels:
    None
  • Environment:
    Ubuntu (Hoary), Apache 2.2, MySQL 5, PHP 5.1.2, Moodle 1.6.1+
  • Affected Branches:
    MOODLE_16_STABLE

Description

I'm not sure where to categorize this as it has to do with the /user directory.

I have had teachers complain that when using the select multiple users via the participants link (\user\index.php -> /user/action_redir.php -> /user/messageselect.php that it is failing. This seems to take an inordinate amount of time to process and I was suspecting that it was timing out; however, now teachers are reporting that even just with a few it is failing. As I went in and looked at the code in /user/messageselect.php I noticed that it only reports an error at the end which makes it difficulty to troubleshoot which user it was not able to send to. I am wondering if the issue is because we have several dummy email user accounts that we created for the students that are disabled. In any case, I had a couple of requests for these pages.

First, the /user/action_redir.php and the /user/messageselect.php files do not have the version header so it is difficulty to know without doing a diff what version is on the server.

Second, should Moodle output a success/failure message for each message that it sends. It seems it could be helpful as it is stepping through the foreach found at line 89 in the \user\messageselect.php. If it is not good then let the user know what failed rather than simply that a failure took place.

Thanks for your consideration of improving this new and useful functionality

Activity

Hide
Anthony Borrow added a comment -

Something as simple as:

if ($good) { echo '<p>Message sent to '$user'</p>'; } else { echo '<p>Error sending message to '.$user'</p>'; }

could be very useful IMO.

Show
Anthony Borrow added a comment - Something as simple as: if ($good) { echo '<p>Message sent to '$user'</p>'; } else { echo '<p>Error sending message to '.$user'</p>'; } could be very useful IMO.

People

Vote (3)
Watch (2)

Dates

  • Created:
    Updated: