Skip to content

BEN-2954: [DO NOT MERGE] Rebase Django-Filer to original Repo #158

Open
andreilupuleasa wants to merge 886 commits into
masterfrom
BEN-2954
Open

BEN-2954: [DO NOT MERGE] Rebase Django-Filer to original Repo #158
andreilupuleasa wants to merge 886 commits into
masterfrom
BEN-2954

Conversation

@andreilupuleasa

Copy link
Copy Markdown
Collaborator

No description provided.

mariasumedre-3pg and others added 30 commits April 16, 2021 16:43
BEN-2235: Proxy all unknown method calls to the queryset
BEN-2247: set content type on file upload
… deleting a folder

In the newer django version (2.2), the MPTTModel.delete() methods now calls django.db.models.base.Model.refresh_from_db, which in turn calls the constructor of the Folder model (django-filer/filer/models/foldermodels.py:Folder). At that moment, almost all of the fields are DEFERRED, trying to acces them will trigger again django.db.models.base.Model.refresh_from_db, thus the infinite loop.

The fix: set the 'cached' props only if fields are fetched from DB.
BEN-2299 fix filer error: maximum recursion depth exceeded  while har…
in the `_copy_file` method, after the actual file is copied in storage in the new path, the old_name is not properly reset, so the method `_is_path_changed` will again try to copy the file in storage, when File obj is saved.
@andreilupuleasa andreilupuleasa changed the title BEN-2954 BEN-2954: Rebase Django-Filer to original Repo Jul 6, 2026
@andreilupuleasa andreilupuleasa changed the base branch from master to release-3.4.4 July 6, 2026 09:25
@andreilupuleasa andreilupuleasa changed the base branch from release-3.4.4 to master July 6, 2026 09:30
@andreilupuleasa andreilupuleasa changed the title BEN-2954: Rebase Django-Filer to original Repo BEN-2954: [DO NOT MERGE] Rebase Django-Filer to original Repo Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants