Url routing error

Hi.

I have problem on my site where urls starting with a digit is failing (404 page). The jReview team was running a debug and says the error is caused by jfbconnect. Could you take a look, and see if there is anything you could fix on your side.

Here is the debug info from the jReview team:

I was finally able to figure out why this doesn't happen on my site, but it does on yours. It's because the problem is caused by the JFBConnect System Plugin, not by the JReviews SEF Plugin. If you disable that plugin then it works. Unfortunately, it's not possible for us to provide a fix for this inside the JReviews SEF Plugin because that means making a compromise that would cause a lot of other issues for the rest of the clients, and this issue is specific to the use of this extension. I have a much older version of JFBConnect installed in localhost and the issue doesn't happen with that old version.

I narrowed down the issue to this line specifically inside the 'onAfterDispatch' event.

ftp://yyxx//plugins/system/jfbcsystem/jfbcsystem.php
Line 153: $app->triggerEvent('onOpenGraphAfterDispatch');

So looking further inside JFBC where that event is defined in the onOpenGraphAfterDispatch method

ftp://yyxx//libraries/sourcecoast/openGraphPlugin.php

The problem is that this line:

$url = $juri->toString();

Returns the URL as you see it in the browser URL and when JFBC tries to get the correct parameters from it, it can't because the JReviews SEF plugin has not run at this point and it thinks that 6 is the article ID.

So the fix would be to have that part of the code run only AFTER all router parse actions have already finished executing. Maybe the JFBC dev knows what he needs to do. In the mean time if you want a quick fix, that may have other negative consequences you can edit this file:

ftp://yyxx//plugins/system/jreviews_sef/jreviews_sef.php

$router->attachParseRule(array($JreviewsRouter, 'parseJReviews'), JRouter::PROCESS_AFTER);

And replace it with:

$router->attachParseRule(array($JreviewsRouter, 'parseJReviews'));

However, we are not able to make this change in our distribution of JReviews, because all the other bugs we fixed related to Joomla 3.6 depend on this.

I tried the fix, but unfortuantly it was not working. I have attaced the error here:

For now i have removed this line, and now i´m able to access url´s with a digit.

I´m sorry to inform you that the bug is still alive after updating to 7.0.4. Any url´s starting with a digit is being redirected to a 404 page.

Hi.

All url starting with a digit:
www.in2city.com/news/6-reasons-to-visit-biograd-na-moru (<- 404 error)
www.in2city.com/index.php?option=com_con...view=article&id=6274 (<- works)

Native Joomla Articles with Jreview (i use jReview for my blog/news articles)

No other SEF plugins then jReview

If i add the quickfix from jReview the url with digits starts working. I also works if i remove this line: $app->triggerEvent('onOpenGraphAfterDispatch');

Let me know if you need access to our site.

Hi Alex,

I had to apply the quickfix from jreview to avoid seo issues. Let me know if you like to revert the changes if you like to run more debug on our site.