Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Hotfixes the dbtransfer/upload to correctly obtain the oldBlobKey #103

Merged
merged 2 commits into from
Mar 25, 2019

Conversation

phorward
Copy link
Member

@phorward phorward commented Oct 8, 2018

No description provided.

@tsteinruecken tsteinruecken self-requested a review October 12, 2018 07:04
dbtransfer.py Outdated
@@ -142,13 +142,17 @@ def upload( self, oldkey, *args, **kwargs ):
"parentdir": "",
"parentrepo": "",
"weak": False } )

oldkey = conf["viur.mainApp"].file.decodeFileName(oldkey)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: "self" is not used in decodeFileName. We could make this function a class method and call it directly (instead of relying that a file module is loaded under conf["viur.mainApp"].file)

@phorward phorward self-assigned this Nov 19, 2018
@sveneberth sveneberth added this to the v2.4 milestone Feb 27, 2019
…e in dbtransfer.py and added a direct import instead
@tsteinruecken tsteinruecken changed the base branch from master to develop March 25, 2019 14:37
@tsteinruecken tsteinruecken merged commit 8e8976c into develop Mar 25, 2019
@tsteinruecken tsteinruecken deleted the fix_dbtransfer.upload branch March 25, 2019 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants