Uploaded image for project: 'Islandora'
  1. Islandora
  2. ISLANDORA-1296

Add $associations to $elements in theme('islandora_solr_metadata_display', $elements)

    Details

    • Type: Code Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Fix Version/s: 7.x-1.6
    • Component/s: Solr Metadata
    • Labels:
      None

      Description

      Currently, islandora_solr_metadata_get_associations_by_cmodels() is called twice, once from within template_preprocess_islandora_solr_metadata_display() and once prior to that in islandora_solr_metadata_display_callback(), where theme('islandora_solr_metadata_display', $elements) is called. If we remove the call to islandora_solr_metadata_get_associations_by_cmodels() from the preprocess function and add $associations to $elements, we get two benefits: 1) one less database call and 2) other modules that don't associate Solr metadata viewer configurations with content models, like Islandora Context, can generate their own $associations to pass into theme('islandora_solr_metadata_display', $elements).

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              markj Mark Jordan
              Reviewer:
              Nick Ruest
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: