This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "FusionForge".

The branch, Branch_5_3 has been updated
       via  7353d7c1a1262d2ee669277dbb6c53a021c9a0b6 (commit)
      from  b50acbd61113ab314ac03029f579552321c6024b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 7353d7c1a1262d2ee669277dbb6c53a021c9a0b6
Author: Franck Villaume <[email protected]>
Date:   Fri May 30 20:42:01 2014 +0200

    fix [#682]: fix search engine documents, use the right section to check 
about permissions

diff --git a/src/CHANGES b/src/CHANGES
index b14284e..1215a0f 100644
--- a/src/CHANGES
+++ b/src/CHANGES
@@ -33,7 +33,8 @@ Fusionforge-5.3.1:
 * Plugin message: handle empty site message (Inria)
 * Plugin mediawiki: fix icon in project page link (Inria)
 * Plugin hudson: fix roles access in widgets [#683] (TrivialDev)
-* Admin User: fix unix_status [#666](TrivialDev)
+* Admin User: fix unix_status [#666] (TrivialDev)
+* Search: fix roles based access to documents using the search engine [#682] 
(TrivialDev)
 
 FusionForge-5.3:
 * Widget: MyArtifacts Enhancement : add monitored artifacts (TrivialDev)
diff --git 
a/src/www/search/include/renderers/DocsAllHtmlSearchRenderer.class.php 
b/src/www/search/include/renderers/DocsAllHtmlSearchRenderer.class.php
index 54dc741..560a296 100644
--- a/src/www/search/include/renderers/DocsAllHtmlSearchRenderer.class.php
+++ b/src/www/search/include/renderers/DocsAllHtmlSearchRenderer.class.php
@@ -4,7 +4,7 @@
  *
  * Copyright 2004 (c) Dominik Haas, GForge Team
  * Copyright 2011, Franck Villaume - Capgemini
- * Copyright 2013, Franck Villaume - TrivialDev
+ * Copyright 2013,2014 Franck Villaume - TrivialDev
  * Copyright 2013, French Ministry of National Education
  * http://fusionforge.org
  *
@@ -87,11 +87,11 @@ class DocsAllHtmlSearchRenderer extends HtmlSearchRenderer {
                $rowColor = 0;
                for($i = 0; $i < $rowsCount; $i++) {
                        //section changed
-                       $currentDocGroup = db_result($result, $i, 
'project_name');
-                       $currentDocGroupObject = 
group_get_object_by_publicname($currentDocGroup);
-                       if(!forge_check_perm('project_read', 
$currentDocGroupObject->getID())) {
+                       if(!forge_check_perm('docman', db_result($result, $i, 
'project_name'), 'read')) {
                                continue;
                        }
+                       $currentDocGroup = db_result($result, $i, 
'project_name');
+                       $currentDocGroupObject = 
group_get_object_by_publicname($currentDocGroup);
                        if ($lastDocGroup != $currentDocGroup) {
                                $return .= 
'<tr><td>'.html_image('ic/home16b.png', '10', '12', array('border' => 
'0')).'<b>'.util_make_link('/docman/?group_id='.$currentDocGroupObject->getID(),$currentDocGroup).'</b></td><td
 colspan="3">&nbsp;</td></tr>';
                                $lastDocGroup = $currentDocGroup;

-----------------------------------------------------------------------

Summary of changes:
 src/CHANGES                                                    |    3 ++-
 .../include/renderers/DocsAllHtmlSearchRenderer.class.php      |    8 ++++----
 2 files changed, 6 insertions(+), 5 deletions(-)


hooks/post-receive
-- 
FusionForge

_______________________________________________
Fusionforge-commits mailing list
[email protected]
http://lists.fusionforge.org/cgi-bin/mailman/listinfo/fusionforge-commits

Reply via email to