View Issue Details

IDProjectCategoryView StatusLast Update
0000167Main CAcert Websiteweb of trustpublic2013-01-14 01:30
Reportersglaser Assigned ToSourcerer  
PrioritynormalSeveritytrivialReproducibilityhave not tried
Status closedResolutionfixed 
Fixed in Version2006 
Summary0000167: No points might be added because the member already has got 35 points
DescriptionWhile assuring a member with 35 points already allocated by someone else, I received a warning that the system might round down my 10 points or even assign no new points at all because of the aforesaid 35 points.
If I didn't get something wrong this message should only appear if the sum of the points I can allocate at most together with the points already allocated exceeds 100.
Additional Information- I was using German as my default language.
- My maximum of 10 points was correctly and immediately allocated.
TagsNo tags attached.
Reviewed by
Test Instructions

Activities

duane

2006-08-16 09:08

developer   ~0000516

Last edited: 2006-08-16 09:08

- if($_SESSION['_config']['pointsalready'] > 0)

+ if(100 - $_SESSION['_config']['pointsalready'] - maxpoints() <= 0)

Sourcerer

2006-08-23 23:14

administrator   ~0000632

Sorry, but this doesn´t solve the issue correctly.

+ if(100 - $_SESSION['_config']['pointsalready'] - maxpoints() <= 0)

1. Super Assurers can issue up to 150 points, so the hardcoded value 100 is definitely wrong.
2. The error message says that the user already has points. Your IF statement triggers also when the user does not have any points.

I think the statement should look something like this:

+ if(($_SESSION['_config']['pointsalready']>0) && (maxmaxpoints() - $_SESSION['_config']['pointsalready'] - maxpoints() <= 0))

duane

2006-09-06 12:14

developer   ~0000663

Fixed.

Sourcerer

2006-09-06 21:57

administrator   ~0000667

Yes, looks fixed to me.

Issue History

Date Modified Username Field Change
2006-03-08 07:30 sglaser New Issue
2006-08-16 09:08 duane Status new => solved?
2006-08-16 09:08 duane Resolution open => fixed
2006-08-16 09:08 duane Assigned To => duane
2006-08-16 09:08 duane Note Added: 0000516
2006-08-16 09:08 duane Note Edited: 0000516
2006-08-23 23:14 Sourcerer Note Added: 0000632
2006-08-23 23:14 Sourcerer Status solved? => @30@
2006-09-06 12:14 duane Status @30@ => needs work
2006-09-06 12:14 duane Assigned To duane => Sourcerer
2006-09-06 12:14 duane Status needs work => solved?
2006-09-06 12:14 duane Fixed in Version => production
2006-09-06 12:14 duane Note Added: 0000663
2006-09-06 21:57 Sourcerer Status solved? => closed
2006-09-06 21:57 Sourcerer Note Added: 0000667
2013-01-14 01:30 Werner Dworak Fixed in Version => 2006