Pull Request for Issue #10140 and #40996

Summary of Changes

• Put article ids in a UserState variable
• Use the article ids to get all the keywords from the articles
• Avoid deprecation Warning in BannersModel

Testing Instructions taken from #10140

-Create a banner, including some meta keywords.
-Create a banner module an setup the option "Search by Meta Keyword" to "YES".
-Activate the module in any position let's say "position-7" on Protostar template
-Disable the cache for the banners module and save.

-Create a Content article and setup some Meta Keywords which will be used to display the banner accordingly.
-Embed the module Banners inside the article using the "Module" button on the tinyMCE editor (the syntax will look something like this {loadmodule mod_banners,banners module title} ) and save it.

Edit: Can be tested with a normal banner module with 'Select by keywords' set to 'Yes'

Edit: Test with Client and No-Client set - both should work

Actual result BEFORE applying this Pull Request

-The Banners module embedded inside the article shows no content at all.
-The source code shows:
< div class = " module " >
< h3 > Banners module title< / h3 >
< div class = " banner group " > < / div >
< / div >

Expected result AFTER applying this Pull Request

-The Banners module works good and shows the banner according to the Meta Keywords in the "position-7" or any other position in the default Protostar template.
-The banners module should also work the same way when embedded inside the article, displaying the banners according to the Meta Keywords.

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed