Merge branch 'seeddms-5.1.x' into seeddms-6.0.x

This commit is contained in:
Uwe Steinmann 2023-10-31 16:17:47 +01:00
commit d7e780493f
2 changed files with 21 additions and 18 deletions

View File

@ -287,6 +287,7 @@
- check if document version may be removed - check if document version may be removed
- set max file size for indexing when updating a document - set max file size for indexing when updating a document
- nicer output on info page - nicer output on info page
- do not show filter for categories in database search if they don't exist
-------------------------------------------------------------------------------- --------------------------------------------------------------------------------
Changes in version 5.1.32 Changes in version 5.1.32

View File

@ -619,20 +619,22 @@ $(document).ready(function() {
$tmpcatids[] = $tmpcat->getID(); $tmpcatids[] = $tmpcat->getID();
$options = array(); $options = array();
$allcategories = $dms->getDocumentCategories(); $allcategories = $dms->getDocumentCategories();
foreach($allcategories as $acategory) { if($allcategories) {
$options[] = array($acategory->getID(), $acategory->getName(), in_array($acategory->getId(), $tmpcatids)); foreach($allcategories as $acategory) {
$options[] = array($acategory->getID(), $acategory->getName(), in_array($acategory->getId(), $tmpcatids));
}
$this->formField(
getMLText("categories"),
array(
'element'=>'select',
'class'=>'chzn-select',
'name'=>'category[]',
'multiple'=>true,
'attributes'=>array(array('data-placeholder', getMLText('select_category'), array('data-no_results_text', getMLText('unknown_document_category')))),
'options'=>$options
)
);
} }
$this->formField(
getMLText("categories"),
array(
'element'=>'select',
'class'=>'chzn-select',
'name'=>'category[]',
'multiple'=>true,
'attributes'=>array(array('data-placeholder', getMLText('select_category'), array('data-no_results_text', getMLText('unknown_document_category')))),
'options'=>$options
)
);
$options = array(); $options = array();
if($workflowmode == 'traditional' || $workflowmode == 'traditional_only_approval') { if($workflowmode == 'traditional' || $workflowmode == 'traditional_only_approval') {
if($workflowmode == 'traditional') { if($workflowmode == 'traditional') {
@ -897,7 +899,7 @@ $(document).ready(function() {
foreach($values as $v=>$c) { foreach($values as $v=>$c) {
$uu = $dms->getUserByLogin($v); $uu = $dms->getUserByLogin($v);
if($uu) { if($uu) {
$option = array($uu->getId(), $v); $option = array($uu->getId(), htmlspecialchars($v));
if(isset(${$facetname}) && in_array($uu->getId(), ${$facetname})) if(isset(${$facetname}) && in_array($uu->getId(), ${$facetname}))
$option[] = true; $option[] = true;
else else
@ -910,7 +912,7 @@ $(document).ready(function() {
foreach($values as $v=>$c) { foreach($values as $v=>$c) {
$cat = $dms->getDocumentCategoryByName($v); $cat = $dms->getDocumentCategoryByName($v);
if($cat) { if($cat) {
$option = array($cat->getId(), $v); $option = array($cat->getId(), htmlspecialchars($v));
if(isset(${$facetname}) && in_array($cat->getId(), ${$facetname})) if(isset(${$facetname}) && in_array($cat->getId(), ${$facetname}))
$option[] = true; $option[] = true;
else else
@ -933,7 +935,7 @@ $(document).ready(function() {
/* Do not even create a list of options, because it isn't used */ /* Do not even create a list of options, because it isn't used */
} else { } else {
foreach($values as $v=>$c) { foreach($values as $v=>$c) {
$option = array($v, $v); $option = array($v, htmlspecialchars($v));
if(isset(${$facetname}) && in_array($v, ${$facetname})) if(isset(${$facetname}) && in_array($v, ${$facetname}))
$option[] = true; $option[] = true;
else else
@ -1141,7 +1143,7 @@ $(document).ready(function() {
else else
$fu = $dms->getUserByLogin($v); $fu = $dms->getUserByLogin($v);
if($fu) if($fu)
$oldtransval[] = $fu->getLogin(); $oldtransval[] = htmlspecialchars($fu->getLogin());
} }
break; break;
case 'status': case 'status':
@ -1309,7 +1311,7 @@ $(document).ready(function() {
case 'owner': case 'owner':
foreach($values as $v=>$c) { foreach($values as $v=>$c) {
if($fu = $dms->getUserByLogin($v)) if($fu = $dms->getUserByLogin($v))
$menuitems[] = array('label'=>$fu->getLogin(), 'link'=>$newrequest->getRequestUri().'&'.$facetname.'[]='.$fu->getId(), 'badge'=>$c); $menuitems[] = array('label'=>htmlspecialchars($fu->getLogin()), 'link'=>$newrequest->getRequestUri().'&'.$facetname.'[]='.$fu->getId(), 'badge'=>$c);
} }
break; break;
default: default: