mahara-contributors team mailing list archive

[Kristina Hoeppner <1743902@xxxxxxxxxxxxxxxxxx>]

** Changed in: mahara
       Status: New => Confirmed

** Tags added: regression

** Changed in: mahara
   Importance: Undecided => High

** Changed in: mahara
    Milestone: None => 18.04.0

** Also affects: mahara/17.10
   Importance: Undecided
       Status: New

** Also affects: mahara/18.04
   Importance: High
       Status: Confirmed

** Changed in: mahara/17.10
       Status: New => Confirmed

** Changed in: mahara/17.10
   Importance: Undecided => High

** Changed in: mahara/17.10
    Milestone: None => 17.10.4

-- 
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:
  Confirmed
Status in Mahara 17.10 series:
  Confirmed
Status in Mahara 18.04 series:
  Confirmed

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