?
avatar coolcat-creations
coolcat-creations
24 Jun 2017

Steps to reproduce the issue

It seems that is has to do with the Change in #11217

I tested local on a customer page and got a 404 on the Home Item (Featured Articles)
I could not reproduce the issue on a clean install, but disabling the pagebreak plugin showed the page like expected.

Seems something in the check is wrong
if (StringHelper::strpos($row->text, 'class="system-pagebreak') === false) { if ($page > 0) { throw new Exception(JText::_('JERROR_PAGE_NOT_FOUND'), 404); } return true; }

avatar coolcat-creations coolcat-creations - open - 24 Jun 2017
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jun 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 24 Jun 2017
avatar zero-24 zero-24 - change - 24 Jun 2017
Labels Added: ?
avatar zero-24 zero-24 - labeled - 24 Jun 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 25 Jun 2017
Category com_content com_plugins
avatar rdeutz rdeutz - change - 25 Jun 2017
Labels Removed: ?
avatar rdeutz rdeutz - unlabeled - 25 Jun 2017
avatar rdeutz
rdeutz - comment - 25 Jun 2017

Without instructions how to reproduce it, we can't do so much. The test seems not wrong.

avatar franz-wohlkoenig franz-wohlkoenig - change - 25 Jun 2017
Status New Discussion
avatar zero-24
zero-24 - comment - 25 Jun 2017

@coolcat-creations it has nothing todo with featured articles can you confirm that on that page a news flash module is published?

@rdeutz here is how to reproduce:

How to reproduce

  • install 3.7.3-dev
  • set up a news flash module to the category Joomla! (using sample data)

result

image

Looks like some other things are broken now too.

Reason

https://github.com/joomla/joomla-cms/blob/3.7.3-rc1/modules/mod_articles_news/helper.php#L124

Because $page is set to 1 and no page break is included this throws the error. But i don't know what the $page var is supposed to do?

Disableling the pagebreak plugin fixes the issue. To me this is a release-blocker as it breaks existing sites.

avatar coolcat-creations
coolcat-creations - comment - 25 Jun 2017

Thank you @zero-24 yes i confirm that unpublishing Newsflash also solves the 404 Error.

avatar zero-24
zero-24 - comment - 25 Jun 2017

@Bakual @laoneo can you take a look here and told us why you set $page === 1 in that module?

aa862e8 && 2d82f4c

avatar rdeutz
rdeutz - comment - 25 Jun 2017

I don't think we need to set $page, I made a patch #16866

avatar laoneo
laoneo - comment - 25 Jun 2017

I guess because it was set with page 1 on the other events, like onContentAfterTitle, which existed already.

avatar zero-24
zero-24 - comment - 25 Jun 2017

Closing as the PR #16866 fixes it. Thanks!

avatar zero-24 zero-24 - change - 25 Jun 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-06-25 18:56:32
Closed_By zero-24
avatar zero-24 zero-24 - close - 25 Jun 2017
avatar eriknzzz
eriknzzz - comment - 7 Jul 2017

I am new to github, but have a problem with pagebreak since Joomla 3.7.3 that may be related to the above discussion.
I am writing articles with extensive use of the pagebreak (using JCE editor). After the upgrade the browser page header shows the current page title with the page number. In the article the page title is added to the article title. Previously the browser page header only showed the article title for any page and the article title was left alone (as it should).
Any suggestions to fix this (until hopefully the Joomla team does by the next upgrade)?

avatar zero-24
zero-24 - comment - 7 Jul 2017

Please open a new issue describing in detail how to reproduce your issue. The issue described in this issue is fixed in 3.7.3 and should not be your problem. Thanks.

avatar conflate-nl
conflate-nl - comment - 25 Jul 2017

This is still not fixed properly, because this breaks any pagination on Joomla articles, if no pagebreaks are used in that article. Simply go to any article page without pagebreak, and add ?limitstart=20 to the url.
We have a custom list which has pagination, and now we get a 404 but the page is just fine, since the list is paginated. After I found that it was the Page Break plugin, I've disabled it, but now I can't use page breaks. A simple switch in the plugin settings if a 404 should be thrown, would solve this nicely and would give other plugins still the use of pagination on Joomla article pages.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 25 Jul 2017

@conflate-nl can you please open a new Issue?

avatar eriknzzz
eriknzzz - comment - 26 Jul 2017

Already done #17127

avatar conflate-nl
conflate-nl - comment - 27 Jul 2017

Actually #17127 is not the problem, the problem is on pages which aren't using page breaks, but custom pagination lists. I've created a new Issue #17305

Add a Comment

Login with GitHub to post a comment