Issue Details (XML | Word | Printable)

Key: MDL-15336
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: Eloy Lafuente (stronk7)
Reporter: Eloy Lafuente (stronk7)
Votes: 0
Watchers: 0
Operations

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

Review SQL CEIL() uses across code. MSSQL uses CEILING()

Created: 21/Jun/08 08:52 AM   Updated: 02/Jul/08 07:04 AM
Return to search
Component/s: Database SQL/XMLDB
Affects Version/s: 1.9.1, 2.0
Fix Version/s: 1.9.2, 2.0

Issue Links:
Relates
 

Database: Any
Participants: Eloy Lafuente (stronk7) and Petr Skoda
Security Level: None
QA Assignee: Petr Skoda
Resolved date: 29/Jun/08
Affected Branches: MOODLE_19_STABLE, MOODLE_20_STABLE
Fixed Branches: MOODLE_19_STABLE, MOODLE_20_STABLE


 Description  « Hide
While fixing and testing MDL-13861, it seems that the statistics generations has some CEIL() SQL uses.

But MSSQL isn't using that function but CEILING() instead.

To fix this:

1) Create one sql_ceil() function.
2) Replace all uses of CEIL() with that function.

Ciao :-)

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Eloy Lafuente (stronk7) added a comment - 29/Jun/08 04:52 AM
Resolving as fixed. New sql_ceil() function added and applied. 19_STABLE and HEAD.

Ciao


Petr Skoda added a comment - 02/Jul/08 07:04 AM
thanks