Project

General

Profile

Bug #7577

Error in Buying course plugin (version 1.9.10 and 1.10)

Added by Andre Boivin over 4 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Category:
Plugins
Target version:
Start date:
06/03/2015
Due date:
% Done:

100%

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

Description

At line 312 in buy_course.lib.php
$sql = "SELECT 1 FROM %s WHERE %s ='" . Database::escape_string($parameter) . "' AND id_user='" . intval($user) . "';";
change for $sql = "SELECT 1 FROM %s WHERE %s ='" . Database::escape_string($parameter) . "' AND user_id='" . intval($user) . "';";

Associated revisions

Revision 7bf4da4b (diff)
Added by Yannick Warnier over 4 years ago

Fix query in buy_course plugin - refs #7577

Revision c7b17060 (diff)
Added by Yannick Warnier over 4 years ago

Fix query in buy_course plugin - refs #7577

History

#1

Updated by Yannick Warnier over 4 years ago

Actually it's a bit more complicated than that (id_user is for session_rel_user but course_rel_user calls it user_id), so I sent the following to 1.10.x:

https://github.com/chamilo/chamilo-lms/commit/7bf4da4b7a2311b901fd2941a28f01096d5ceb6d

and this to 1.9.x:

https://github.com/chamilo/chamilo-lms/commit/c7b17060a77ea57961bfc168690b4d0312c60d39

#2

Updated by Yannick Warnier over 4 years ago

  • Status changed from New to Needs testing
  • Assignee set to Andre Boivin
  • % Done changed from 0 to 100

The plugin was apparently broken in 1.9.x. Sorry for that. We did a lot of testing but apparently it wasn't enough for production.
Thanks for the report.

#3

Updated by Yannick Warnier over 4 years ago

  • Target version set to 1.10.0
#4

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee changed from Andre Boivin to Yannick Warnier

Also available in: Atom PDF