Moodle

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

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.9.1, 2.0
  • Fix Version/s: 1.9.2, 2.0
  • Component/s: Database SQL/XMLDB
  • Labels:
    None
  • Database:
    Any
  • Affected Branches:
    MOODLE_19_STABLE, MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE, MOODLE_20_STABLE

Description

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

Issue Links

Activity

Hide
Eloy Lafuente (stronk7) added a comment -

Resolving as fixed. New sql_ceil() function added and applied. 19_STABLE and HEAD.

Ciao

Show
Eloy Lafuente (stronk7) added a comment - Resolving as fixed. New sql_ceil() function added and applied. 19_STABLE and HEAD. Ciao
Hide
Petr Škoda (skodak) added a comment -

thanks

Show
Petr Škoda (skodak) added a comment - thanks

People

Dates

  • Created:
    Updated:
    Resolved: