TP-Docs
HTML5 Icon HTML5 Icon HTML5 Icon
TP on Social Media

Recent

Welcome to TinyPortal. Please login or sign up.

Members
  • Total Members: 3,963
  • Latest: BiZaJe
Stats
  • Total Posts: 195,910
  • Total Topics: 21,307
  • Online today: 872
  • Online ever: 8,223 (February 19, 2025, 04:35:35 AM)
Users Online
  • Users: 0
  • Guests: 365
  • Total: 365

Article Ratings Error

Started by clothahump, February 20, 2011, 12:50:59 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

lurkalot

TBH, they'll probably be in the next release.  But IchBin has provided the fix for you to try above.  Hopefully it'll sort your problem for now.  ;)

clothahump

I was just thinking of people downloading the current version and then having to trawl through the forum to find the fixes, surely the current version could be updated every time an error is fixed?

WillyP

No, the reason for knowing version numbers is so we know whats in the code.  If you start releasing updates without any way to track them, you end up with chaos.  We have trouble enough getting people to state the version when asking for help, imagine having to ask for them to dig in the code to see if random changes had been made.  When a number of small patches have been made, Ich will release a new version.  If it was a critical security patch, I am sure he would release a new version right away.  But this is a minor feature that does not affect the overall operation of the forum.

IchBin

Generally if I post the fix in the topic, I post it in the bugtracker too. If I don't post the fix, it's because there are too many changes, or it's a change I wouldn't want someone to try just yet. 

Lurk, my guess is that these things have changed over the last while with Bloc moving things, and updating them. I haven't looked, but I'd say that ratings may have been broken longer than just TP1RC1. Either way, we are getting closer. The biggest issues to solve are the front page problems with articles/pagination, and Agridocs language issues.

clothahump


IchBin

Thanks for reporting that clothahump. I forgot about that section. I'll dig into this when I can get access back on my server. :)

IchBin

If you'd like to try a fix for this one too you can try this. I don't know if it works, so make sure you backup your file before making these changes.
Sources/TPortal.php about line #1056
// expand the vislaoptions
$row['visual_options'] = explode(",", $row['options']);


Add this right before the lines found above:
// Add the rating together
$row['rating'] = array_sum(explode(',', $row['rating']));

clothahump

Fine, that one worked as well, by the time we are finished it will be a new release. :)

This website is proudly hosted on Crocweb Cloud Website Hosting.