Update doc string,image alt tex and issue creation

src/pe_reports/__init__.py

@@ -47,7 +47,7 @@
 db = SQLAlchemy(app)
 Migrate(app, db)
 
-# TODO: Add a login page
+# TODO: Add a login page in the future. Issue #207 contains details
 # login_manager.init_app(app)
 # login_manager.login_view = "login"
 
@@ -56,7 +56,7 @@
 
 # Register the flask apps
 app.register_blueprint(stakeholder_blueprint)
-# TODO: Add login blueprint
+# TODO: Add login blueprint. Issue #207 contains details
 # app.register_blueprint(manage_login_blueprint)
 app.register_blueprint(home_blueprint)
 

src/pe_reports/stakeholder/forms.py

@@ -10,7 +10,7 @@ class InfoFormExternal(FlaskForm):
     """Create web form to take user input on organization information/details."""
 
     cust = StringField("What is the stakeholder name?", validators=[DataRequired()])
-    # TODO: The following form field may be used in a future update
+    # TODO: The following form field may be used in a future update. Issue #208
     # custIP = StringField(
     #     "What is the stakeholder ip/cidr? *comma separate entries",
     #     validators=[DataRequired()],
@@ -21,7 +21,7 @@ class InfoFormExternal(FlaskForm):
     custDomainAliases = StringField(
         "What are the organization aliases? " "*comma separate entries"
     )
-    # TODO: The following form field may be used in a future update
+    # TODO: The following form field may be used in a future update. Issue #208
     # custSubDomain = StringField(
     #     "What is the sub-domain for this stakeholder?" " *comma separate entries"
     # )

src/pe_reports/stakeholder/views.py

@@ -51,7 +51,7 @@ def getToken():
 def getAgencies(org_name):
     """Get all agency names from P&E database."""
     global conn, cursor
-    resultDict = {}
+
     try:
         params = config()
 
@@ -71,6 +71,7 @@ def getAgencies(org_name):
             cursor.execute(query.format(org_name))
 
             result = cursor.fetchall()
+            resultDict = {}
 
             for row in result:
                 # row[0] = org UUID
@@ -90,7 +91,7 @@ def getAgencies(org_name):
 
 
 def getRootID(org_UUID):
-    """Get all agency names from P&E database."""
+    """Get all root domain names from P&E database."""
     global conn, cursor
     resultDict = {}
     try:
@@ -164,7 +165,7 @@ def setStakeholder(customer):
 
 
 def setCustRootDomain(customer, rootdomain, orgUUID):
-    """Insert customer into the PE-Reports database."""
+    """Insert customer root domain into the PE-Reports database."""
     global conn, cursor
 
     try:
@@ -439,7 +440,7 @@ def setNewCSGOrg(newOrgName, orgAliases, orgdomainNames, orgIP, orgExecs):
     newOrgID = response["id"]
 
     if newOrgID:
-        logging.info("Got here there is a new new org %s", newOrgID)
+        logging.info("A new org_id was created: %s", newOrgID)
 
         setOrganizationUsers(newOrgID)
         setOrganizationDetails(newOrgID, orgAliases, orgdomainNames, orgIP, orgExecs)

src/pe_reports/templates/home.html

@@ -3,7 +3,7 @@
 <div class="container">
   <img
     src="{{ url_for('static', filename='CISAImage.png') }}"
-    alt="Cinque Terre"
+    alt="CISA seal image"
     width="200"
     height="200"
   />