User tests: Successful: Unsuccessful:
Pull Request for Issue #45570
This PR fixes an issue where Joomla allows the creation of multiple banners with the same alias within the same category, despite the code intending to enforce alias uniqueness.
Previously, the alias uniqueness check existed only within the update functionality of the store function in the BannerTable
class, meaning it was bypassed during insert operations. This PR moves the uniqueness validation outside of the update logic so that it runs consistently during both insert and update operations.
I agree with Brian's comment on the initial issue though, I'm not fully sure what the alias is actually used for.
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
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_banners |
Labels |
Added:
PR-5.3-dev
|
I am wondering if it makes sense to have the same banner (name and alias) for different clients?
That's a good question, and I'm really not sure. It doesn't appear there's anything from stopping 2 banners from having the same name.
As far as the alias, I'm really not sure that the alias is actually used for anything. I can't find anything it's used for on the component or the banner module, but it could be something someone added as a plan for the future? I tested out the links it generates for tracking clicks and none of that uses an alias it seems at the moment, it's all pretty much ID based.
Labels |
Added:
bug
|
I am wondering if it makes sense to have the same banner (name and alias) for different clients?