TOPIC: link to article wrong with sh404sef in mod_comments

Hi! I have a problem with mod_comments. The link to article is not working if i have sh404sef. It redirects to ..../.html?...... instead of the right path to the article.

It seems that on line 194 of
/administrator/components/com_comment/plugin/com_content/josc_com_content.class.php
the contentId variable should only need to be numeric, without the alias. If i comment that line, like below, it works again.

function linkToContent($contentId, $commentId='', $joscclean=false, $admin=false, $dbg=false) {
$appl =& JFactory::getApplication();

if ($appl->scope == 'mod_comments') {
$mod_data = $this->getCatSecAliasMod($contentId);
$this->_route_catid = $mod_data->catid;
$this->_route_sectionid = $mod_data->sectionid;
//$contentId = $contentId . '-' . $mod_data->alias;
}

Aha - I suppose this bug only happens if "file suffix" in sh404SEF is blank. mod_comments assumes that it is set to html.

This is a little beyond my knowledge so we'll wait for Daniels' feedback on this - obviously there is a bug but I think we need a way to check sh404SEF "file suffix" option so the module generates correct links.

Thanks again Mario!

sh404SEF expect numeric value - that can be the reason why when I use it on localhost it crashes! Oh my god!

K, I will check this on Thursday and let you know how it goes.

hi, i have the same problem in my site. Does there any solution to solve this wrong link in the module mod_comments ?
thx

I was having the same problem, but after I commented out that line, it seems to work. Thanks for the help!

I remember that i've added the alias variable only because the module was not generating friendly URLs with joomla's default SEF system turned on. I will have to test it with 404sef. Thanks for the help guys!

can you tell me exactly how to uncomment that line?

Do I need to delete it?

Do I need to delete this whole code?

function linkToContent($contentId, $commentId='', $joscclean=false, $admin=false, $dbg=false) {
$appl =& JFactory::getApplication();

if ($appl->scope == 'mod_comments') {
$mod_data = $this->getCatSecAliasMod($contentId);
$this->_route_catid = $mod_data->catid;
$this->_route_sectionid = $mod_data->sectionid;
//$contentId = $contentId . '-' . $mod_data->alias;
}

uncommenting is just placing // at the begining of the line.

So from:it should become:

to add the // at the beginning of a line is to "comment-out" the code, uncomment would be to remove the // comment marker

good point and you are absolutely right. I will think a little bit more next time

The fault is very forgivable - as I said before, your English is better than many Australians I know! I also think you are a little (un)lucky for getting a language major as a developer/helper for JoomlaComment

always a good idea to search first, ask questions later -- I had this problem with content comments @ Kalamies.com but now fixed.

:thumbs-up-to-those-who-solved-this-smiley:

Well, this doesn't solve it for me because, either way, the links generated point towards http://domain/index.php/

Any suggestion?

Thanks

That is strange. Please check out how the new module is working for you Robin.

Thanks Daniel,

I will check it out and let you know of the outcome.

Robin

OK, it works with the new versions (component + module). I found, however, a new bug (I will post it o a new thread).

Thanks,
Robin