No Code Attached Yet
avatar cozioscob
cozioscob
19 Jun 2023

Problem identified

Banner component & module 'Select By Keyword' not working. e.g. adding keyword 'all' to an article and adding the same keyword to a banner module and related banner results in no banner showing.
I believe this is due to article keywords functionality being removed as it's no longer necessary for SEO. However this is necessary for the Banner component keyword targetting to behave correctly. Can we reinstate some form of keyword functionality?
Keywords 'are' working for related articles functionality which is promising, just not for banners.

Proposed solution

Reinstate keyword functionality for banners

Open questions

Is this possible?

avatar cozioscob cozioscob - open - 19 Jun 2023
avatar joomla-cms-bot joomla-cms-bot - change - 19 Jun 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Jun 2023
avatar brianteeman
brianteeman - comment - 19 Jun 2023
avatar richard67 richard67 - change - 19 Jun 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-06-19 16:24:36
Closed_By richard67
avatar richard67 richard67 - close - 19 Jun 2023
avatar richard67
richard67 - comment - 19 Jun 2023

Closing as having a pull request. Please test #39016 . Thanks in advance.

avatar cozioscob
cozioscob - comment - 19 Jun 2023

#39016 Not working for me either on my live site running 4.3.2 or a fresh install of 4.3.2 or 4.4.0-alpha2-dev. e.g. here
Turning off Select By Keyword in the banner module shows banner, turning on (with keyword 'test' setup in the article and for the banner)

avatar MacJoom
MacJoom - comment - 19 Jun 2023

@cozioscob #39016 has not been merged yet - you could test this which would advance the process

avatar cozioscob
cozioscob - comment - 19 Jun 2023

@cozioscob #39016 has not been merged yet - you could test this which would advance the process

To be clear, I've:

  1. Replaced BannersModel.php, ArticleModel.php, ArticlesModel.php, SiteApplication.php, BannersHelper.php from 39016 on my working site, no success (in fact it broke my site layout)

  2. Replaced BannersModel.php, ArticleModel.php, ArticlesModel.php, SiteApplication.php, BannersHelper.php in a fresh install of 4.3.2 with no change to banner visibility using keyword setup

  3. Replaced BannersModel.php, ArticleModel.php, ArticlesModel.php, SiteApplication.php, BannersHelper.php in a fresh install of 4.4.0-alpha2-dev with no change to banner visibility using keyword setup

avatar MacJoom
MacJoom - comment - 20 Jun 2023

@cozioscob You can try with the packages from here: https://ci.joomla.org/artifacts/joomla/joomla-cms/4.3-dev/39016/downloads/66712/ - i tested this and it works fine.

avatar cozioscob
cozioscob - comment - 20 Jun 2023

Managed to work out the issue.
If Client is set to 'No Client' in the module then no banners will display, choosing a client works with the above and keywords enabled.

Error with no client selected is
Warning: Undefined array key 4 in ----/public_html/components/com_banners/src/Model/BannersModel.php on line 231

Using 'No Client' is my preferred as it then pulls in banners from all clients and uses keywords to prioritise banners related to several different clients.

Hope that helps?
(I'm also seeing a 1064 error on my home page only which disappears if I unpublish the banner modules from home, might be related)

avatar MacJoom
MacJoom - comment - 21 Jun 2023

Thank you very much - could you please open a new issue for the "No client" problem - i can then look into it. A screenshot of the 1064 error would also be helpful

avatar cozioscob
cozioscob - comment - 21 Jun 2023

Here you go. #40996

Add a Comment

Login with GitHub to post a comment