owncloud / qnap

App for qnap - license defines number of allowed users - this app disables those over the limit
GNU General Public License v2.0
3 stars 3 forks source link

get users in a sorted order so that users will be deactivated in a de… #39

Closed wkloucek closed 3 years ago

wkloucek commented 3 years ago

…terministic way

fixes https://github.com/owncloud/qnap/issues/33

trigger order by by setting offset to 0 instead of null

https://github.com/owncloud/core/blob/77c629b64d7bd84fbd4907df63597c475b0c778e/lib/private/User/AccountMapper.php#L236-L238

codecov[bot] commented 3 years ago

Codecov Report

Merging #39 (8c28347) into master (ebd1385) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #39   +/-   ##
=========================================
  Coverage     77.18%   77.18%           
  Complexity       69       69           
=========================================
  Files             7        7           
  Lines           206      206           
=========================================
  Hits            159      159           
  Misses           47       47           
Impacted Files Coverage Δ
lib/Command/CheckActiveUsers.php 88.37% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ebd1385...8c28347. Read the comment docs.