Hello Plus Header and Footer Widgets Site Title / Logo should link to Home URL and not to Site URL #36395
Mihuer
started this conversation in
Feature Request
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Describe the Problem
The PHP function "get_site_url()" in the Plugin File ".../hello-plus/modules/template-parts/widgets/ehp-widget-base.php" returns the Site Link in both EHP-Footer and EHP-Header as Site URL (site_url()) instead of Home URL (home_url()). This is not ideal as this returns the path to the Wordpress-Installation instead of the Homepage. If the installation is in a Folder, for instance yoursite.com/wordpress/, but your home url is yoursite.com, you get a 404. If a site has multiple languages, this is also a problem, as the site URL does not translate, while the Home URL does by default (with WPML or others), for instance yoursite.com/de/ .
Propose a Solution
It requires to change only one line of code in the file "ehp-widget-base.php" (line 548), which I tested successfully as following:

Additional Context
I would be great if this could be considered in the next release of the plugin. It's totally risk free because for those with no problems with that feature (site url and home url are the same), nothing will actually change.
Agreement
All reactions