Inclusion of analytics code wrong?

Support Forums for LiveMesh Themes & Plugins Forums Fusion Theme Support Inclusion of analytics code wrong?

Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
    Posts
  • #15607
    GroupXSgroupXS
    Member

    Hi there,

    we were wondering why our google analytics code was not working. Then we noticed that the option strips out the <script>. Looking at the code we found that in the page the code was not surrounded by <script> when rendered in the page.

    So we changed this:

    diff ./framework/extensions/framework-extender.php,orig ./framework/extensions/framework-extender.php
    386c386
    < echo ‘< div class=”hidden”>’ . $analytics_code . ‘< /div>’;

    > echo ‘<script>’ . $analytics_code . ‘</script>’;

    (I hope this is readable to you, I had to change it a bit because of the editor of this site. Basically this should be a diff where we replaced the div tags with script tags surrounding the
    $analytics_code)

    Hope that is correct?

    Cheers
    Tobias

    • This topic was modified 5 years, 8 months ago by GroupXSgroupXS. Reason: Ugly code
    • This topic was modified 5 years, 8 months ago by GroupXSgroupXS.
    • This topic was modified 5 years, 8 months ago by GroupXSgroupXS.
    #15630
    RaghavendraRaghavendra
    Moderator

    Actually we had to cripple this functionality in our Fusion theme due to security concerns raised by ThemeForest theme review team. It is no longer allowed for users to enter scripts in theme options or other settings.

    Pls use a plugin like this one –

    https://wordpress.org/plugins/google-analytics-for-wordpress/

    We will have this option removed from the theme options in future update of the theme.

    #15643
    GroupXSgroupXS
    Member

    Aha, that explains it. Thank you for the reply. I would urge you to put a notification about this issue somewhere. It took me about 1h to understand why our tracking code did not work.

    #15653
    RaghavendraRaghavendra
    Moderator

    Sure and will remove this option asap. Sorry for the trouble.

Viewing 4 posts - 1 through 4 (of 4 total)
  • The forum ‘Fusion Theme Support’ is closed to new topics and replies.