Moodle

Browse list of users not working with MSSQL

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7
  • Fix Version/s: 1.7
  • Component/s: Administration
  • Labels:
    None
  • Environment:
    Moodle 1.7 beta with MSSQL 2005. Both database and Apache/PHP running on same windows XP box.
    Using freetds to get to MSSQL
  • Database:
    Microsoft SQL
  • Affected Branches:
    MOODLE_17_STABLE
  • Fixed Branches:
    MOODLE_17_STABLE

Description

  • Fresh install of Moodle 1.7 beta on SQL 2005
  • Add 1 or 2 users
  • Go to http://localhost/admin/user.php and select "Browse a list of users"
  • Resulting page does not show actual users (though count of users is correct). INstead you get the message: "No users matching ' were found"

Probable cause: I traced the SQL with SQL Profiler and one of the queries fauils because it containts the "LIMIT" keyword (which mssql does not support). The offending query is:

SELECT id, username, email, firstname, lastname, city, country, lastaccess, confirmed
FROM mdl_user
WHERE deleted <> '1' AND username <> N'changeme' ORDER BY firstname ASC LIMIT 0,30

The full .sql file capturing the sql output when you select 'browe users' is attached.

Issue Links

Activity

Hide
Eloy Lafuente (stronk7) added a comment -

Hi Chris,

If I'm not wrong this should be working since 1-2 days ago. Both in MDL-7167 and MDL-7168 we replaced all the LIMIT clauses present in Moodle Core to their cross-db alternative (using PHP ADOdb API to decide the proper QUERY to be executed for each DB).

So, if you upgrade your Moodle installation to the last Beta code available at http://download.moodle.org (updated daily) and the issue should be solved. Note that changes these days are happening really quick so, perhaps it would be interesting to you to use direct CVS access to upgrade whenever you want (http://docs.moodle.org/en/CVS_for_Administrators)

Thanks for the report!

Show
Eloy Lafuente (stronk7) added a comment - Hi Chris, If I'm not wrong this should be working since 1-2 days ago. Both in MDL-7167 and MDL-7168 we replaced all the LIMIT clauses present in Moodle Core to their cross-db alternative (using PHP ADOdb API to decide the proper QUERY to be executed for each DB). So, if you upgrade your Moodle installation to the last Beta code available at http://download.moodle.org (updated daily) and the issue should be solved. Note that changes these days are happening really quick so, perhaps it would be interesting to you to use direct CVS access to upgrade whenever you want (http://docs.moodle.org/en/CVS_for_Administrators) Thanks for the report!
Hide
Chris Moffatt added a comment -

Hi Eloy,

I confirmed that this problem works in latest sources - and will test against the most current sources

Show
Chris Moffatt added a comment - Hi Eloy, I confirmed that this problem works in latest sources - and will test against the most current sources
Hide
Eloy Lafuente (stronk7) added a comment -

Great!

Definitively closing this now.

Show
Eloy Lafuente (stronk7) added a comment - Great! Definitively closing this now.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: