Skip to content

[6.0] Remove deprecated db object from views#42962

Merged
wilsonge merged 6 commits intojoomla:6.0-devfrom
Digital-Peak:view/db
Nov 2, 2024
Merged

[6.0] Remove deprecated db object from views#42962
wilsonge merged 6 commits intojoomla:6.0-devfrom
Digital-Peak:view/db

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented Mar 5, 2024

Summary of Changes

Removes some deprecated database properties in views. They stayed to not break overrides during the upgrade from 3 to 4. But are not used anymore.

Testing Instructions

  • Create an article
  • Open the article on the front tend

Actual result BEFORE applying this Pull Request

All works.

Expected result AFTER applying this Pull Request

All works.

Link to documentations

Please select:

@brianteeman
Copy link
Contributor

Should this be documented ?

@laoneo
Copy link
Member Author

laoneo commented Mar 19, 2024

Yes, just didn't had time.

@laoneo laoneo added b/c break This item changes the behavior in an incompatible why. HEADS UP and removed Documentation Required labels Mar 26, 2024
@laoneo
Copy link
Member Author

laoneo commented Mar 26, 2024

Added documentation

@HLeithner HLeithner changed the title Remove deprecated db object from views [6.0] Remove deprecated db object from views Apr 24, 2024
@Hackwar
Copy link
Member

Hackwar commented Oct 23, 2024

Can you fix the codestyle issues?

@laoneo
Copy link
Member Author

laoneo commented Oct 23, 2024

done

@Quy
Copy link
Contributor

Quy commented Nov 1, 2024

I have tested this item ✅ successfully on 0b2603a


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

1 similar comment
@ghost
Copy link

ghost commented Nov 2, 2024

I have tested this item ✅ successfully on 0b2603a


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

@Quy
Copy link
Contributor

Quy commented Nov 2, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 2, 2024
@wilsonge wilsonge merged commit b7cd314 into joomla:6.0-dev Nov 2, 2024
@wilsonge wilsonge deleted the view/db branch November 2, 2024 12:42
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 2, 2024
@wilsonge
Copy link
Contributor

wilsonge commented Nov 2, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

b/c break This item changes the behavior in an incompatible why. HEADS UP Feature PR-6.0-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants