github: shadowy octopus with the head of a robot, emblazoned with the Dreamwidth swirl (Default)
github ([personal profile] github) wrote in [site community profile] changelog2022-11-18 05:24 pm

[dreamwidth/dreamwidth] 72326a: limit directory search by location results to visi...

Branch: refs/heads/main Home: https://github.com/dreamwidth/dreamwidth Commit: 72326a5b4e8ccaddcb8732eaf28c02fa57b0a43e https://github.com/dreamwidth/dreamwidth/commit/72326a5b4e8ccaddcb8732eaf28c02fa57b0a43e Author: Kareila kareila@dreamwidth.org Date: 2022-11-15 (Tue, 15 Nov 2022)

Changed paths: M cgi-bin/LJ/Directory/Constraint/Location.pm

Log Message:


limit directory search by location results to visible users

I took my cue from ContactInfo.pm, which mentions a desire to avoid using load_userids on a large data set for filtering purposes. Instead this expresses the visibility requirement as an additional WHERE clause in the SELECT statement, acting only on users that have rows in both the user and userprop tables.

Obviously my ability to test this locally is extremely limited, but this should fix the problem with location results being sparsely populated due to a large number of suspended SEO accounts.

Commit: 3390d019706aacbe797f3b21d1a38e55d236b11d https://github.com/dreamwidth/dreamwidth/commit/3390d019706aacbe797f3b21d1a38e55d236b11d Author: momijizukamori momijizukamori+bugzilla@gmail.com Date: 2022-11-18 (Fri, 18 Nov 2022)

Changed paths: M cgi-bin/LJ/Directory/Constraint/Location.pm

Log Message:


Merge pull request #3024 from kareila/location-results

limit directory search by location results to visible users

Compare: https://github.com/dreamwidth/dreamwidth/compare/2547f60cfc8d...3390d019706a