Skip to content

[5.4] Replace deprecation AbstractView::get() in layouts#45702

Merged
muhme merged 3 commits intojoomla:5.4-devfrom
heelc29:5.4/deprecations/layouts
Jul 26, 2025
Merged

[5.4] Replace deprecation AbstractView::get() in layouts#45702
muhme merged 3 commits intojoomla:5.4-devfrom
heelc29:5.4/deprecations/layouts

Conversation

@heelc29
Copy link
Contributor

@heelc29 heelc29 commented Jul 7, 2025

Summary of Changes

remove deprecated access to AbstractView::get() for layout files

@Hackwar as negative side effect #44162 the access to protected properties of the view class is no longer possible. Is that acceptable?

if (isset($this->$property)) {
return $this->$property;
}

Testing Instructions

check views still working eg.

  • banner edit view (backend)
  • admin module edit view (backend)
  • categories view of contact, content, newsfeeds (frontend)
  • category view of contact, content, newsfeeds (frontend)

Actual result BEFORE applying this Pull Request

deprecated access to method get of AbstractView

Expected result AFTER applying this Pull Request

no deprecated access to method get of AbstractView

Link to documentations

Please select:

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

@exlemor
Copy link

exlemor commented Jul 12, 2025

I have tested this item ✅ successfully on 60d1379

I have tested this successfully.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45702.

@heelc29 heelc29 marked this pull request as ready for review July 14, 2025 16:45
@heelc29 heelc29 changed the title [5.4] Remove deprecation AbstractView::get() in layouts [5.4] Replace deprecation AbstractView::get() in layouts Jul 24, 2025
@hans2103
Copy link
Contributor

I have tested this item ✅ successfully on 60d1379


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45702.

@richard67
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45702.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 25, 2025
@muhme
Copy link
Contributor

muhme commented Jul 26, 2025

Final test before merge, before applying this PR, with JBT graft Joomla_5.4.0-alpha4-dev-Development-Full_Package.zip, used links are listed – IDs needs to be picked-up from backend:

  • Renamed all uncategorised to Content-Uncategorised, Banners-Uncategorised and Contacts-Uncategorised
  • Created one article, one banner, one contact and one news feed entry
  • Banner edit view (backend)
    • Components > Banners > Banners > First Banner
  • Admin module edit view (backend)
    • System > Administrator Modules > Sample Data
  • Categories view of contact, content, newsfeeds (frontend w/o login)
    • /index.php?option=com_contact&view=categories
    • /index.php?option=com_content&view=categories
    • /index.php?option=com_newsfeeds&view=categories
  • Category view of contact, content, newsfeeds (frontend w/o login)
    • /index.php?option=com_contact&view=category&id=4
    • /index.php?option=com_content&view=category&id=2
    • /index.php?option=com_newsfeeds&view=category&id=5
      • ⚠️ Page contains a broken link to the /index.php/component/newsfeeds/newsfeed/first-news-feed?catid=5

Tested with this PR by graft Joomla_5.4.0-alpha4-dev+pr.45702-Development-Full_Package.zip:

  • ✅ Tested all the backend views, including editing
  • ✅ Tested all the frontend views, including the links in the views
  • ✅ The previous broken link is now working

@muhme muhme merged commit f1906ba into joomla:5.4-dev Jul 26, 2025
32 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 26, 2025
@muhme muhme added this to the Joomla! 5.4.0 milestone Jul 26, 2025
@muhme
Copy link
Contributor

muhme commented Jul 26, 2025

Thank you @heelc29 for your contribution and @Hackwar for your code review. Thank you @exlemor and @hans2103 for testing.

@heelc29 heelc29 deleted the 5.4/deprecations/layouts branch August 7, 2025 16:34
@brianteeman
Copy link
Contributor

This has broken com_guidedtours so clearly it is a b/c break which should not happen in a minor release and must be reverted and moved to j6 see #45938

brianteeman added a commit to brianteeman/joomla-cms that referenced this pull request Aug 19, 2025
muhme pushed a commit that referenced this pull request Aug 28, 2025
* Revert "[5.4] Replace deprecation AbstractView::get() in layouts (#45702)"
* This reverts commit f1906ba.
softforge pushed a commit that referenced this pull request Aug 28, 2025
* [5.3] Translation Update (#45983)

* Fix copy / paste error (#45979)

* [5.3] Security updates for composer and npm dependencies for the upcoming 5.3.4 release (#45984)

* composer update enshrined/svg-sanitize to 0.22.0

* npm fix audit issues

* [5.3] Update TinyMCE from 6.8.5 to 6.8.6 to fix TinyMCE issue with cursor placement (#45987)

* npm update tinymce from 6.8.5 to 6.8.6

* Update version in tinymce.xml

* [5.4] Revert b/c breaking change in AbstractView::get (#45940)

* Revert "[5.4] Replace deprecation AbstractView::get() in layouts (#45702)"
* This reverts commit f1906ba.

* [5.3] Update joomla/filesystem to fix extension uploads when post_max_size is 0 (#45986)

* [5.4] Upmerge 2025-08-28 (#45997)

* [5.3] Translation Update #45983
* [5.3] Fix copy / paste error in form fields #45979
* [5.3] Security updates for composer and npm dependencies for the upcoming 5.3.4 release #45984
* [5.3] Update joomla/filesystem to fix extension uploads when post_max_size is 0 #45986

* [5.4] Composer update joomla/filesystem to 3.2.0 (#45999)

* Update content-hash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants