Skip to content

Commit

Permalink
Fix Package.should_have_pointer_file()
Browse files Browse the repository at this point in the history
Before this, this method was incorrectly returning `True` if the package
was an AIP (or AIC) even if it was not a file (i.e., was not
compressed).
  • Loading branch information
jrwdunham committed Nov 29, 2017
1 parent 6c70bd8 commit 23d1e07
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion storage_service/locations/models/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -561,7 +561,7 @@ def should_have_pointer_file(self, package_full_path=None,
package_type = self.package_type
isfile = os.path.isfile(package_full_path)
isaip = package_type in (Package.AIP, Package.AIC)
ret = isfile or isaip
ret = isfile and isaip
if not ret:
if not isfile:
LOGGER.info('Package should not have a pointer file because %s'
Expand Down

0 comments on commit 23d1e07

Please sign in to comment.