Skip to content

Commit 6c12410

Browse files
Removes analytics_property setting from theme
** Why are these changes being introduced: * Initially I was thinking that the analytics property ID would be a configurable value for each site, so I'd built a settings field for that value. However, we've since clarified that this will not be optional, or configurable. ** Relevant ticket(s): * https://mitlibraries.atlassian.net/browse/post-36 ** How does this address that need: * This removes the analytics_property settings field from the theme, including how it was passed to the FooterHelper as an example of a string to be processed. The FooterHelper itself is being removed in a parallel PR to this one, so I'm leaving it in the footer. ** Document any side effects to this change: * None, hopefully.
1 parent d299eee commit 6c12410

2 files changed

Lines changed: 1 addition & 7 deletions

File tree

config/theme.ini

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,6 @@ omeka_version_constraint = "^4.0.0"
1010
helpers[] = "FooterHelper"
1111

1212
[config]
13-
elements.analytics_property.name = "analytics_property"
14-
elements.analytics_property.type = "Text"
15-
elements.analytics_property.options.label = "Analytics Property ID"
16-
elements.analytics_property.options.info = "This is the Matomo property ID, and can be provided by UXWS"
17-
1813
elements.banner.name = "banner"
1914
elements.banner.type = "Omeka\Form\Element\Asset"
2015
elements.banner.options.label = "Site banner"

view/layout/layout.phtml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
<?php
22
$translate = $this->plugin('translate');
3-
$analytics_property = $this->themeSetting('analytics_property');
43
$banner_url = $this->themeSettingAssetUrl('banner');
54
$banner_text = $this->themeSetting('banner_text');
65

@@ -98,7 +97,7 @@ $userBar = $this->userBar();
9897
<?php echo $this->partial('common/footer-site-slim'); ?>
9998
</div>
10099
<footer>
101-
<?php echo $this->FooterHelper($analytics_property); ?>
100+
<?php echo $this->FooterHelper(''); ?>
102101
<span class="powered-by"><?php echo $translate('Powered by Omeka S'); ?></span>
103102
</footer>
104103
</body>

0 commit comments

Comments
 (0)