Joomla Components and Professional Joomla Templates

      
    | 

13th of March 2010

Joomla Backend Mastery - FREE webinar - find out more here.
Welcome, Guest
Please Login or Register.    Lost Password?

Error in latest version of Module
(1 viewing) (1) Guest
Go to bottomPage: 12
TOPIC: Error in latest version of Module
#8812
Error in latest version of Module 1 Month, 1 Week ago Karma: 0
I've been having problems with the Module giving incorrect links (back to the home page rather than article) when listing most commented. I'm using sh404sef and tried all the fixes mentioned on the board without luck.

I then got digging in the code and found there is an error in line 80 of mod_comments.class.php

It should read
$id = $row['contentid'];
not
$id = $row['id'];


I hope this helps.
Andrew
Fresh Boarder
Posts: 10
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#8829
Re: Error in latest version of Module 1 Month, 1 Week ago Karma: 68
which joomlacomment version are you using 4.0RC1 ?
Daniel Dimitrov
Developers, developers, developers :)
Administrator
Posts: 3073
graph
User Offline Click here to see the profile of this user
Gender: Male Location: Karlsruhe, Germany Birthday: 05/06
The administrator has disabled public write access.
Please write a review for joomlacomment at extensions.joomla.org

When you send me more info through the PM system, please provide a link to your problem in the forum!
 
#8833
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 0
Yes, but this error is in the module. I'm using a recent version of the module that you provided on the forum in response to as question.

I checked the contents of the array being passed to the module and it didn't include id, just contentid.

I've tested the fix and it works.
Andrew
Fresh Boarder
Posts: 10
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#8837
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 68
If you look carefully at the module code you will find out that we use code from the component. That is why I asked you if you use RC1.
Daniel Dimitrov
Developers, developers, developers :)
Administrator
Posts: 3073
graph
User Offline Click here to see the profile of this user
Gender: Male Location: Karlsruhe, Germany Birthday: 05/06
The administrator has disabled public write access.
Please write a review for joomlacomment at extensions.joomla.org

When you send me more info through the PM system, please provide a link to your problem in the forum!
 
#8840
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 0
I realise that. I know that the link is formed in josc_com_content.class.php. But it is the module that wasn't working because the information wasn't being passed correctly.

I do like your product though and I'm posting this to help not to criticise.
Andrew
Fresh Boarder
Posts: 10
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#8841
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 68
Hey Andrew,
I'm not assuming that you are criticizing - I just want to know which version of joomlacomment you are using. I can not remember all the things I've changed to the component. So perhaps this is something solved in RC1. This is why I am asking you if you use the latest version or not.
Daniel Dimitrov
Developers, developers, developers :)
Administrator
Posts: 3073
graph
User Offline Click here to see the profile of this user
Gender: Male Location: Karlsruhe, Germany Birthday: 05/06
The administrator has disabled public write access.
Please write a review for joomlacomment at extensions.joomla.org

When you send me more info through the PM system, please provide a link to your problem in the forum!
 
#8842
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 0
Sorry, my initial answer was unclear. I did say yes to using RC1 but I can see that wasn't obvious.
Andrew
Fresh Boarder
Posts: 10
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#8844
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 68
Ah good. I'll check it out then, but I have this strange feeling that if I make your change, that it won't work for all other plugins
Daniel Dimitrov
Developers, developers, developers :)
Administrator
Posts: 3073
graph
User Offline Click here to see the profile of this user
Gender: Male Location: Karlsruhe, Germany Birthday: 05/06
The administrator has disabled public write access.
Please write a review for joomlacomment at extensions.joomla.org

When you send me more info through the PM system, please provide a link to your problem in the forum!
 
#8847
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 0
But the change is only to the module, it doesn't alter the component.

But yes, knock-on effects would not be good!
Andrew
Fresh Boarder
Posts: 10
graphgraph
User Offline Click here to see the profile of this user
The administrator has disabled public write access.
 
#8851
Re:Error in latest version of Module 1 Month, 1 Week ago Karma: 12
Forgive me if I missed something, but this makes no sense to me. contentid returns Undefined Index error and the entire module breaks. Please tell us your Joomla version, PHP version and MySQL version, and if you have ever installed any other commenting components as there may be lingering (yComments, JomComment, JComments, etc)

Also tell us if you have done any upgrading on this site (Joomla 1.0 > 1.5, or JoomlaComment 3.26 > 4.0, etc.)

Daniel did you actually try the "fix" in the original post?

EDIT: Do you have one of the outdated versions of sh404SEF installed (all the ones that were free) or are you up to date (paid versions)? If you're up to date please make a screen dump of your settings and I will compare to mine, I can put this in a FAQ about JoomlaComment or sh404SEF.
JonusC
Moderator
Posts: 792
graphgraph
User Offline Click here to see the profile of this user
Last Edit: 2010/02/10 00:42 By jonusc.
The administrator has disabled public write access.
Scripting for fun ;D
 
Go to topPage: 12
Moderators: Daniel Dimitrov, JonusC

Our sponsors

Who's online

We have 925 guests and 7 members online

Support us!

Our Blog

Joomla Backend Mastery - free Webinar! (23)
09.03.2010, 07:57:
I had a look over compojoom's forum last days. There are always some people that ask obvious questions in my eyes: "The comments are not showing" - we ...
Read more

Advertisement