Project

General

Profile

Bug #8089

Impossible to post a message in a social group

Added by Yannick Warnier almost 4 years ago. Updated almost 4 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Social
Target version:
Start date:
15/02/2016
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

  • create a social group
  • write a message in the discussions section:

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE23000: Integrity constraint violation: 1048 Column 'dest_user_id' cannot be null' in /var/www/chamilo110x/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php on line 112
( ! ) PDOException: SQLSTATE23000: Integrity constraint violation: 1048 Column 'dest_user_id' cannot be null in /var/www/chamilo110x/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php on line 91

There is probably a FK on dest_user_id in the message table, and there shouldn't be (because when sending a message in a social group, there is no specific user destination).

Associated revisions

Revision 896f3fb1 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Set the correct value for params arrays in MessageManager::send_message - refs #8089

Revision 9d7be8c3 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Set the correct value for params arrays in MessageManager::send_message - refs #8089

Revision 506c4928 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Fix creating message on social group - refs #8089

Revision 586f2826 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Fix creating message on social group - refs #8089

Revision aabd8a29 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Add Behat tests for social group - refs #8089

Revision 561367f9 (diff)
Added by Angel Fernando Quiroz Campos almost 4 years ago

Add Behat tests for social group - refs #8089

History

#1

Updated by Yannick Warnier almost 4 years ago

The message gets created (surprisingly), but the error clearly is an issue.

#2

Updated by Angel Quiroz almost 4 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Angel Quiroz)
  • % Done changed from 0 to 90

Fixed!
Applied also on master

#3

Updated by Yannick Warnier almost 4 years ago

  • Assignee set to Yannick Warnier
#4

Updated by Yannick Warnier almost 4 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee changed from Yannick Warnier to Angel Quiroz
  • % Done changed from 90 to 100

Tested and approved.
Good job on adding the Behat tests!

Also available in: Atom PDF