Issue Details (XML | Word | Printable)

Key: MDL-6605
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Petr Skoda
Reporter: Penny Leach
Votes: 1
Watchers: 6
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

switch datalib functions (wherever they are now) to use placeholders

Created: 20/Sep/06 04:57 AM   Updated: 06/Jul/08 01:07 AM
Component/s: Lib
Affects Version/s: None
Fix Version/s: 2.0

File Attachments: 1. HTML File skypelog.html (17 kB)

Issue Links:
Dependency
 
Relates
 

Database: Any
Participants: Dan Poltawski, Martin Dougiamas, Martín Langhoff, Penny Leach, Petr Skoda and Tim Hunt
Security Level: None
QA Assignee: Eloy Lafuente (stronk7)
Resolved date: 06/Jul/08
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
change all the get_records and insert/update/ etc functions to use placeholders.
invert magic_quotes_gpc hack to stripslashes rather than addslashes
take away all calls to addslashes

this was already done once (http://git.catalyst.net.nz/gitweb?p=elgg.git;a=commitdiff;h=553765dbdba08162745fe10710ac20153f3c12d2) but will need to be re-done.

All calls to *_sql will need to be migrated, although we can retain backwards compatibility by using $values=null in the function definition and only do variable substitution where $values is not empty.

this DOES break backwards compatibility in a few functinon definitions, see get_records_select in above diff.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
No commits have yet been performed on this issue.