Skip to content

Commit 7ff9e7b

Browse files
authored
Remove reload after saving the batch-edits so the user can handle it themselves, if required. (#1247)
Signed-off-by: Anagh Kumar Baranwal <[email protected]>
1 parent b63bf7f commit 7ff9e7b

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

plexapi/base.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -626,7 +626,8 @@ def batchEdits(self):
626626
return self
627627

628628
def saveEdits(self):
629-
""" Save all the batch edits and automatically reload the object.
629+
""" Save all the batch edits. The object needs to be reloaded manually,
630+
if required.
630631
See :func:`~plexapi.base.PlexPartialObject.batchEdits` for details.
631632
"""
632633
if not isinstance(self._edits, dict):
@@ -635,7 +636,7 @@ def saveEdits(self):
635636
edits = self._edits
636637
self._edits = None
637638
self._edit(**edits)
638-
return self.reload()
639+
return self
639640

640641
def refresh(self):
641642
""" Refreshing a Library or individual item causes the metadata for the item to be

tests/test_video.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -614,7 +614,7 @@ def test_video_Movie_batchEdits(movie):
614614
.editTagline(new_tagline) \
615615
.editStudio(new_studio)
616616
assert movie._edits != {}
617-
movie.saveEdits()
617+
movie.saveEdits().reload()
618618
assert movie._edits is None
619619
assert movie.title == new_title
620620
assert movie.summary == new_summary
@@ -626,7 +626,7 @@ def test_video_Movie_batchEdits(movie):
626626
.editSummary(summary, locked=False) \
627627
.editTagline(tagline, locked=False) \
628628
.editStudio(studio, locked=False) \
629-
.saveEdits()
629+
.saveEdits().reload()
630630
assert movie.title == title
631631
assert movie.summary == summary
632632
assert movie.tagline == tagline

0 commit comments

Comments
 (0)