Respond to code review feedback

Author: matt-bernhardt

config/theme.ini

@@ -7,8 +7,12 @@ theme_link = "https://github.com/MITLibraries/mitlibraries-theme-omeka"
 author_link = "https://libraries.mit.edu"
 omeka_version_constraint = "^4.0.0"
 
+# Helpers are described in the Omeka documentation at
+# https://omeka.org/s/docs/developer/themes/theme_functions/
 helpers[] = ResourceClasses
 
+# Settings are described in the Omeka documentation at
+# https://omeka.org/s/docs/developer/themes/theme_settings/
 [config]
 element_groups.branding = "Site branding settings"
 element_groups.browse = "Site browse settings"
@@ -79,6 +83,8 @@ elements.sites_page.options.element_group = "homepage"
 elements.sites_page.options.label = "Page slug that displays filterable site list"
 elements.sites_page.options.info = "Optional. If provided, this is where the filterable list of sites across the network should be displayed."
 
+# Resource page blocks are described in the Omeka documentation at
+# https://omeka.org/s/docs/developer/themes/theme_use_resource_page_blocks/
 resource_page_blocks.items.main[] = "values"
 resource_page_blocks.items.main[] = "itemSets"
 resource_page_blocks.items.main[] = "sitePages"

helper/ResourceClasses.php

@@ -7,7 +7,9 @@ class ResourceClasses extends AbstractHelper
 {
 
     /**
-     * Returns a representation of resource classes in HTML.
+     * Returns a representation of resource classes in HTML. If this helper
+     * receives anything other than a resource, with a class, which itself has
+     * an ID, then it returns an empty string.
      *
      * @param object $resource The resource to add tags to.
      * @return string This is a set of HTML.