The Platform is a long dead concept, yet its version strings continue to exist in the libraries in @since
and @deprecated
tags. Now that we are talking about CMS 3.x, CMS 4.0, Framework 1.x, and Framework 2.0, having yet another random range of version strings just makes things even more confusing (especially when you get into classes where something has a @since 11.1
tag with a @deprecated 4.0
tag).
Lets get these tags back near the "right" values and get rid of some of the confusion. General rule of thumb that can be used for some bulk updates:
Joomla\CMS\Version
so there shouldn't be anything you'd need to check on that version branch)Labels |
Added:
?
|
Status | New | ⇒ | Discussion |
Category | ⇒ | Language & Strings Libraries |
Good idea
do you want this done for J3 and J4 or just J4?
Both would be nice. Unless you really think the since versions on https://api.joomla.org/cms-3/classes/Joomla.CMS.Language.LanguageHelper.html make sense.
Ok - if no one beats me to it I will put it on my todo list for the sprint on saturday
I have identified the following platform versions in use with J3. Because they are sol old I am struggling to locate exactly which joomla cms versions they should be changed to. Any help appreciated
11.1/2 - Can be anything 1.7.0 or older
11.3 was merged into 1.7.3
11.4 was merged into 2.5.0
12.1 was merged into 3.0.0
12.2 was merged into 3.0.1
12.3 I don't believe was ever actually merged to the CMS, or if it was at that point we were cherry-picking things
13.1 was merged into 3.2.0
There was never a 13.3 so to do it "right" you'd have to follow the git blame back to the right issue or release.
there are 1667 instances of 12.3 ;)
I just looked at the version history for the JPlatform class. It is entirely possible things were cherry-picked and that class never updated, or all the 12.3 things landed with 13.1 in 3.2.0. Or, people just fudged stuff because who cares.
This is what I will be changing to
Status | Discussion | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-10-14 13:14:33 |
Closed_By | ⇒ | brianteeman |
Agree