Merge pull request #211 from mishawakerman/issue-210-restore-options-temp-dir

gabor-boros · web-flow · commit 1f854b1b071f · 2020-06-15T09:47:31.000+02:00
fix: access option.temp_dir correctly
diff --git a/rethinkdb/_restore.py b/rethinkdb/_restore.py
@@ -180,7 +180,7 @@ def parse_options(argv, prog=None):
                 "Temporary directory doesn't exist or is not a directory: %s"
                 % options.temp_dir
             )
-        if not os.access(options["temp_dir"], os.W_OK):
+        if not os.access(options.temp_dir, os.W_OK):
             parser.error("Temporary directory inaccessible: %s" % options.temp_dir)
 
     # - create_args

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ def parse_options(argv, prog=None):`
`180`	`180`	`"Temporary directory doesn't exist or is not a directory: %s"`
`181`	`181`	`% options.temp_dir`
`182`	`182`	`)`
`183`		`- if not os.access(options["temp_dir"], os.W_OK):`
	`183`	`+ if not os.access(options.temp_dir, os.W_OK):`
`184`	`184`	`parser.error("Temporary directory inaccessible: %s" % options.temp_dir)`
`185`	`185`
`186`	`186`	`# - create_args`