pagure

Clone

#5475 fix: Crash when config:[ENABLE_DOCS = False]
Merged a year ago by wombelix. Opened a year ago by wombelix.
wombelix/pagure fix_5407_crash_when_config_enable_docs_eq_false  into  master

file modified
+4 
@@ -33,6 +33,10 @@ 
 

      # from config and we add them here.
 

      if config["ENABLE_DOCS"]:
 

          config["DOCS_FOLDER"] = os.path.join(config["GIT_FOLDER"], "docs")
 

+     else:
 

+         config[
 

+             "DOCS_FOLDER"
 

+         ] = None  # Avoid 'KeyError' Exception down the line
 

      if config["ENABLE_TICKETS"]:
 

          config["TICKETS_FOLDER"] = os.path.join(
 

              config["GIT_FOLDER"], "tickets"

Ensure config["DOCS_FOLDER"] is always defined to avoid KeyError Exception down the line

Fixes: https://pagure.io/pagure/issue/5407

pretty please pagure-ci rebuild
a year ago

rebased onto 66b3e20
a year ago

Metadata Update from @wombelix:
- Request assigned
a year ago

Jenkins seem broken, local tests were fine, minor change without huge risk, I'll merge it to include it in 5.14

Pull-Request has been merged by wombelix
a year ago
Metadata
No Tags
Changes Summary 1
+4 -0
file changed
pagure/config.py
Powered by Pagure 5.14.1
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.