Pull Request resolves #46880

Summary of Changes

This PR enhances the getItem method in com_banners to include state and publishing date validation. By adding these conditions to the database query, we ensure that banner data (including the click URL) is only retrieved if the banner is currently active.

Technical Changes:

Added $nowDate using Factory::getDate()->toSql() for precise timing.

Extended the WHERE clause to check for state = 1.

Added logic to respect publish_up and publish_down fields (including NULL checks).

Testing Instructions

1. Identify a banner ID that is currently unpublished or expired.
2. Attempt to trigger a click via the standard routing: index.php?option=com_banners&task=click&id=[ID].
3. Before PR: The system still retrieves the banner object and increments the click counter.
4. After PR: The getItem method returns no result for inactive banners, effectively preventing the click-tracking task from executing.
5. Verify that published banners within their date range continue to track clicks correctly.

Actual result BEFORE applying this Pull Request

The getItem method retrieves banner details regardless of the publication status, allowing the click-tracking logic to increment counters for inactive ads. This results in "ghost clicks" visible in reporting tools like Looker Studio, where clicks are recorded despite 0 impressions.

Expected result AFTER applying this Pull Request

Banner data is only returned if the banner is published and within the valid date range. Inactive banners will no longer have their clicks tracked.

Link to documentations

• No documentation changes for guide.joomla.org needed
• No documentation changes for manual.joomla.org needed