[#5503] - Routing: Legacy Router Rule for com_newsfeeds
- Closed
- 2 Aug 2015
- Medium
- Build: staging
- # 5503
- Diff
- Hackwar:com_newsfeed_router_legacy
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This implements the current routers behavior of com_newsfeeds as a legacyrouter. This is meant for those that want to keep the old behavior at all costs. Later, parameters will be introduced to switch between the legacy behavior and new features.
This router is copied verbatim from the current router to make sure that the same behavior is kept. No changes should be done to it until Joomla 4.0, when this is supposed to be deleted. The component routers right now are so fragile, that a slow transition will be next to impossible. To prevent any breaks in backwards compatibility, this legacy router is introduced, so that people having problems with the new router can switch back.
How to test
- Check your site before applying the patch and see the URLs of com_newsfeeds.
- Apply the patch.
- See the URLs of com_newsfeeds to stay the same.
Dependencies
This PR requires #5446 to be accepted by the project before it can be applied.
This was made possible through the generous donation of the people mentioned in the following link via an Indiegogo campaign: http://joomlager.de/crowdfunding/5-contributors
jissues-bot
- | Labels |
Added:
?
|
||
brianteeman
- | Category | ⇒ | Router / SEF |
The first URL that you get is from a menu item in the system that links to that category. The second has no menu item. Yes, the URL is not correct, but that is the current behavior. These rules are only there to provide the current behavior and nothing more. They are indeed broken.
joomla-cms-bot
- | Labels |
Added:
?
|
||
This has been updated to work with the latest implementation of JComponentRouterView. Please test.
Same here, looks like you rebased/merged using the 3.5 branch instead of the staging branch?
No, this is expected. The original PR was from a time where no 3.5-dev branch existed. Since the basic classes were merged into the 3.5-dev branch, I had to pull in the 3.5-dev branch into this one to make everything work. Since we can't switch branches for PRs after they have been created, I can't correct that. But overall, this is perfectly fine this way.
If you want to do the PR against the 3.5 branch, you can close this PR and create a new one. You can do that from the existing branch. Doing it like you did doesn't work.
#5444 was against staging and was merged by you guys into 3.5-dev. The other PRs are depending on that code and thus I need to merge that code into this PR. I did discuss this with George. I know how to use git and this here was the reason why I proposed to use a different branching scheme than the current one.
If we were to create a branch for the current minor release and let people create bugfixes against that branch instead of creating a branch for the NEXT minor release and thus force ALL PRs that are meant to go into a minor release to be closed and re-opened or manually merged, we could develope the future version of Joomla in staging and bug-releases would really just be bug-releases, because only serious issues/regressions would be solved by PRs that are targeted at the current minor release branch. But the way it is now, a feature that I'm working on (like for example the routing) would either have to be aimed at staging and then create this mess that we have right now, or you would have to open it against the next minor release branch, then see it not getting merged, close it, open it against the then next minor release branch, again not seeing it being merged, closing, re-opening and then hoping that people will finally get this.
Your PRs are quite a special case.as they depend on eachother. We really have those seldom.
@wilsonge I leave that and #5502 for you to deal with.
As it is now, it's impossible to test and/or review properly as it's targeting another branch than it's based on. Esentially when merging this we would merge 3.5-dev into staging.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-02 16:22:03 |
| Closed_By | ⇒ | Hackwar |
Tested successfully - the URLs stay the same. But how are the URLs built? In testing environment i have "Sample Data Newsfeed" Category but the Url of this item is "/news-feed-category". I created a "Testcat" which url is built to "new-feed-categories/78-testcat" the url of the "Sample Data Newsfeed" seems not clean for me in each case ?! Anyone?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5503.