django-cms / django-filer

File and Image Management Application for django
https://django-filer.readthedocs.io/
Other
1.76k stars 577 forks source link

Fix: str(obj) must not return a lazy object #1405

Closed fsbraun closed 1 year ago

fsbraun commented 1 year ago

Description

This fixes two bugs in the permission object:

Thanks to @prepstarr for pointing out those issues in #1404 and on slack.

Related resources

Checklist

codecov[bot] commented 1 year ago

Codecov Report

Merging #1405 (b3e4347) into master (7315e2b) will increase coverage by 0.02%. The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master    #1405      +/-   ##
==========================================
+ Coverage   74.81%   74.84%   +0.02%     
==========================================
  Files          75       75              
  Lines        3446     3450       +4     
  Branches      554      554              
==========================================
+ Hits         2578     2582       +4     
  Misses        698      698              
  Partials      170      170              
Files Changed Coverage Δ
filer/models/foldermodels.py 88.44% <33.33%> (+0.05%) :arrow_up:
filer/admin/fileadmin.py 94.50% <100.00%> (+0.18%) :arrow_up: