LetoDMS Community Forum
multiple reviewers bug - Printable Version

+- LetoDMS Community Forum (https://community.letodms.com)
+-- Forum: LetoDMS Support (https://community.letodms.com/forumdisplay.php?fid=4)
+--- Forum: Bugs/Errors (https://community.letodms.com/forumdisplay.php?fid=11)
+--- Thread: multiple reviewers bug (/showthread.php?tid=347)



multiple reviewers bug - broebel - 06-03-2011

I'm playing with letoDMS and I really like it sofar!

I found one little issue with multiple reviewers. It is 100% reproducible.
Platform is WAMP.
LetoDMS v 3.0.0 downloaded yesterday.

log in as admin, then:

1) create a new group
2) add admin to this group
3) create user test1 and add to group
4) create user test2 and add to group
5) create a new folder
6) upload a new document
7) assign test1 and test2 as reviewers
8) log in as test1
9) click on newly added document

observe that both reviewers feature an active 'submit review' link, whereas only the one for test1 should be active.


RE: multiple reviewers bug - Doudoux - 06-06-2011

Hy

try this:

Change in out/out.ViewDocument.php file, line 270
PHP Code:
if ($is_reviewer && $status["status"]==S_DRAFT_REV) { 

PHP Code:
if (($required==$user)&&$is_reviewer && $status["status"]==S_DRAFT_REV) { 


doudoux


RE: multiple reviewers bug - kotikov_a - 06-06-2011

(06-06-2011, 03:17 PM)Doudoux Wrote: Hy

try this:

Change in out/out.ViewDocument.php file, line 270
PHP Code:
if ($is_reviewer && $status["status"]==S_DRAFT_REV) { 

PHP Code:
if (($required==$user)&&$is_reviewer && $status["status"]==S_DRAFT_REV) { 


doudoux

Will this modification be in NEW DMS version?


RE: multiple reviewers bug - Doudoux - 06-06-2011

Hi

Once the correction is confirmed by the development team, it should be included in a future version of letoDMS.

doudoux


RE: multiple reviewers bug - broebel - 06-06-2011

(06-06-2011, 03:17 PM)Doudoux Wrote: Hy

try this:

Change in out/out.ViewDocument.php file, line 270
PHP Code:
if ($is_reviewer && $status["status"]==S_DRAFT_REV) { 

PHP Code:
if (($required==$user)&&$is_reviewer && $status["status"]==S_DRAFT_REV) { 


doudoux

Thank you.
I modified the code according to your suggestion and it seems to fix the problem.



RE: multiple reviewers bug - steinm - 07-21-2011

(06-06-2011, 09:10 PM)Doudoux Wrote: Hi

Once the correction is confirmed by the development team, it should be included in a future version of letoDMS.

doudoux

I just added it to the svn. The next stable release will have it.

Uwe


RE: multiple reviewers bug - jknito - 02-08-2012

Hi

this work for one use reviewer, but dont work for a group reviewer.

For grouop reviewer dont allow put review