Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Several fixes #41

Open
wants to merge 72 commits into
base: master
Choose a base branch
from
Open

Several fixes #41

wants to merge 72 commits into from

Conversation

jbreu
Copy link

@jbreu jbreu commented Feb 1, 2017

No description provided.

Galixte and others added 30 commits February 27, 2015 19:03
FRENCH translation for exceptions.php file
FRENCH translation for info_acp_reputation.php
FRENCH translation for permissions_reputation.php
FRENCH translation for reputation_acp.php
FRENCH translation for reputation_common.php
FRENCH translation for reputation_rating.php
FRENCH translation for reputation_system.php
FRENCH translation for reputation_toplist.php
FRENCH translation for reputation_warning.php
FRENCH translation for info_acp_reputation.php file
FRENCH translation for permissions_reputation.php file
FRENCH translation for reputation_acp.php file
FRENCH translation for reputation_common.php file
FRENCH translation for reputation_rating.php file
FRENCH translation for reputation_system.php file
FRENCH translation for reputation_toplist.php file
FRENCH translation for reputation_warning.php file
Update for FRENCH translation for info_acp_reputation.php file
Update for ENGLISH translation for info_acp_reputation.php file
Update for FRENCH translation for reputation_acp.php file
Minor update for ENGLISH for reputation_acp.php file
Update for reputation_common.php file : Change a language key which already exists (duplicate) in reputation_system.php
Update for reputation_common.php file which already exists (duplicate) in reputation_system.php
@jbreu
Copy link
Author

jbreu commented Feb 2, 2017

Thanks for the reports!

@jbreu jbreu closed this Feb 2, 2017
@jbreu jbreu reopened this Feb 2, 2017
@SpeakerCZ
Copy link

In the case of a positive evaluation, the reputation points of the total available quantity are deducted; in the case of a negative evaluation, reputational points are added to the total available quantity.

An example of a positive rating
Reputation points 10 out of a total of 10
I rate a post or user positively with 5 reputation points
The balance is 5 points out of a total of 10

An example of a negative rating
Reputation points 10 out of a total of 10
I evaluate the post or user negatively with 5 reputation points
The balance is 15 points out of a total of 10

But should not 5 points out of 10 ?

@Galixte
Copy link

Galixte commented Apr 20, 2017

Hi @SpeakerCZ,

i use this extension. I didn’t notice this behavior on my board, you can test it, here: http://www.ezcom-fr.com (select the EN flag if your are not french).

Maybe your settings are not corrects (coherent). If you want, i can share mine in english with screenshots.

@Galixte
Copy link

Galixte commented Apr 20, 2017

@eeeman1, i suspect an other extension which is related with avatars that’s the cause of your trouble.

@SpeakerCZ
Copy link

Hello Galixte,
thank you for being a member of your forum. I'm already registered. We have sent you a private message.

@Galixte
Copy link

Galixte commented Apr 20, 2017

Ok, i found the same issue like you.

I think the problem is around this part of code: https://github.com/Pico/phpBB-Reputation-System/blob/master/controller/rating_controller.php#L781

The problme is the subtraction https://github.com/Pico/phpBB-Reputation-System/blob/master/controller/rating_controller.php#L758

$voting_power_left = $max_voting_power - $used_power;

Which means:
voting power (A) = maximum allowed voting power (B) - voting power used (C)

BUT it should calculate like this: A = B - C even if =C is negative, for example:

  • if B = 20 and C = 5, so:
    A = B - C ==> OK A = 15
  • if B = 20 and C = -5, so:
    A = B - (-C) ==> NOT OK A = 25 it should = 15

It takes into account the negative value and adds this with the subtraction thus -- = +.

Any solution @jbreu or @WorkingWarrior ?

@jbreu
Copy link
Author

jbreu commented Apr 21, 2017

I never used that feature. But reading your comments a quick solution seems to be to use the absolute numbers instead of the actual pos/neg values: A = B - abs(C)

@bobalo2
Copy link

bobalo2 commented Jun 8, 2017

Ugh , I'm confused. How do I get this to work with phpbb3.2 ? Can I download the files that were fixed or do I have to change them one by one?

@jbreu
Copy link
Author

jbreu commented Jun 8, 2017

you may want to download one of the current forks: https://github.com/Pico/phpBB-Reputation-System/network

@Galixte
Copy link

Galixte commented Jun 8, 2017

For phpBB 3.2.x the last version is here: https://github.com/jbreu/phpBB-Reputation-System/archive/master.zip.

@Fredhoob
Copy link

Fredhoob commented Apr 10, 2019

I've been using the 1.1.0-dev pico version on my forum for a while. Could I just replace the files with yours and keep all my previous settings, votes of my members, etc.? Thanks.

@jbreu
Copy link
Author

jbreu commented Apr 10, 2019

It should work, but a backup of all data is always recommended

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet