Internal Search is inadvertently retrieving incorrect active user value

Bug #1743902 reported by Ghada El-Zoghbi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Confirmed
High
Unassigned
17.10
Confirmed
High
Unassigned
18.04
Confirmed
High
Unassigned

Bug Description

Mahara: 17.10.2
OS: Linux 16.04
DB: Postgres
Browser: FF

This commit: 210a98b41a10a08409596f731fb6b59d6479ad1c
has inadvertently introduced a bug.

/htdocs/search/internal/lib.php::admin_search_user()

The SQL query to retrieve user data has two columns with the same name 'active'.

One comes from the 'usr' table. The other comes from 'auth_instance' table.

The 'auth_instance.active' column is overwriting the 'usr.active' column because it's listed second.

The issue manifests itself in the user search screen where inactive users are no longer identified as inactive - because the auth_instance is active.

Tags: regression
Changed in mahara:
status: New → Confirmed
tags: added: regression
Changed in mahara:
importance: Undecided → High
milestone: none → 18.04.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.