Case 2
The results are not correct. When you are not logged, you shouldn’t get the document #64 as result. This is an issue.
This issue is due to the fact that, when the user is not logged, I don’t check that the document is or not in the documents_group table.
This is clearly an issue.
The error come from this part of the sql statement:
I need to test sc.privatemgr field too >:(
So for a quick fix of this issue, replace classes/search.class.inc.php code:
// documents should be public
$this->main['filters'][] = array(
'field' => 'privateweb',
'oper' => '=',
'value' => '0'
);
by
// documents should be public
$this->main['filters'][] = array(
'field' => 'privateweb',
'oper' => '=',
'value' => '0'
);
$this->main['filters'][] = array(
'field' => 'privatemgr',
'oper' => '=',
'value' => '0'
);
To conclude, I have change the doc #70 (added proud in the text) and link this document with a new manager group doc "admDocGrp2".
The admDocGrp2 is not linked to the "web" web user group.
I have fixed the release 1.8.2 with the above fix. So you could see the difference now between the release 1.8.1 and 1.8.2.
Search "proud" . You get 2 results with 1.8.1 (one wrong because linked to a manager group) and only 1 (the public) with 1.8.2
I give you some time to see the difference and then I will come back to the original documents (without "proud" on page) and original permissions.
Again, thks for your feedback. I will registered this issue and provide the correction with the 1.8.2
