?
avatar csthomas
csthomas
2 Sep 2017

Steps to reproduce the issue

Install staging Joomla and create an article with 'readmore' and several pages.
Go to the article:

  • page 1 - OK
  • page 2 - error 404

Expected result

All pages should be available

Actual result

Only first one and show all is visible. Other pages show 404

System information (as much as possible)

Staging Joomla

Additional comments

A reason of issue is merged PR #17175.
Plugin pagebreak is called twice.

avatar csthomas csthomas - open - 2 Sep 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 2 Sep 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Sep 2017

looks similar to #17305

avatar franz-wohlkoenig franz-wohlkoenig - change - 2 Sep 2017
Status New Discussion
avatar sandewt
sandewt - comment - 2 Sep 2017

I can 'only' reproduce this issue in combination with a Read More in an article.
One 'Page Break' is sufficient to show a 404 page.

issue17830

avatar sandewt
sandewt - comment - 2 Sep 2017

See file: ..\joomla-cms-staging\plugins\content\pagebreak\pagebreak.php

The code (line 76 etc.) is loading twice.

// Simple performance check to determine whether bot should process further.
if (StringHelper::strpos($row->text, 'class="system-pagebreak') === false)
{
	if ($page > 0)
	{
		throw new Exception(JText::_('JERROR_PAGE_NOT_FOUND'), 404);
	}
	        return true;
}

The first time var_dump(StringHelper::strpos($row->text, 'class="system-pagebreak')) let's see an int(1234) and the second time a bool(false).
This generates the throw, an 404 error page, in line 81.

Testing without Read More shows the first and the second time an int(1234). Stringhelper also loading twice. There is NOT generated an error page.

Why are loading twice?

avatar csthomas
csthomas - comment - 2 Sep 2017

I can 'only' reproduce this issue in combination with a Read More in an article.

Yes. I tested with 'Read More' too.
Ping to @laoneo

avatar csthomas csthomas - change - 20 Sep 2017
The description was changed
avatar csthomas csthomas - edited - 20 Sep 2017
avatar niklasalbin
niklasalbin - comment - 21 Sep 2017

tested reverting PR #17175. It resolves the issue.

avatar AlexRed
AlexRed - comment - 22 Sep 2017

I can confirm the problem in Joomla 3.8.0


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

avatar csthomas
csthomas - comment - 22 Sep 2017

This issue should be fixed by #18066, so I close it.

avatar csthomas csthomas - change - 22 Sep 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-09-22 14:17:25
Closed_By csthomas
avatar csthomas csthomas - close - 22 Sep 2017

Add a Comment

Login with GitHub to post a comment