← Back to team overview

mahara-contributors team mailing list archive

[Bug 1743902] [NEW] Internal Search is inadvertently retrieving incorrect active user value

 

Public bug reported:

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.

** Affects: mahara
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1743902

Title:
  Internal Search is inadvertently retrieving incorrect active user
  value

Status in Mahara:
  New

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.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1743902/+subscriptions


Follow ups