commit c774d418e1caf44e0248bd9d1bcf1ec49abec6be
parent 8fb2ff46ad976b7fe58f471c8e17ca99b290e7da
Author: Jan Dankert <develop@jandankert.de>
Date: Wed, 11 Nov 2020 20:52:26 +0100
Fix: Advanced view for folder was broken.
Diffstat:
5 files changed, 116 insertions(+), 173 deletions(-)
diff --git a/modules/cms/action/FolderAction.class.php b/modules/cms/action/FolderAction.class.php
@@ -1035,7 +1035,7 @@ class FolderAction extends ObjectAction
{
$f = new Folder( $id );
if ( $f->hasRight( Acl::ACL_WRITE ) )
- $otherfolder[$id] = FILE_SEP.implode( FILE_SEP,$f->parentObjectNames(false,true) );
+ $otherfolder[$id] = Startup::FILE_SEP.implode( Startup::FILE_SEP,$f->parentObjectNames(false,true) );
}
asort( $otherfolder );
diff --git a/modules/cms/base/Startup.class.php b/modules/cms/base/Startup.class.php
@@ -30,6 +30,7 @@ class Startup {
const IMG_EXT = '.gif';
const IMG_ICON_EXT = '.png';
+ const FILE_SEP = '/';
const HTML_MODULES_DIR = './modules/';
const THEMES_DIR = './modules/cms/ui/themes/';
diff --git a/modules/cms/ui/themes/default/html/views/folder/advanced.php b/modules/cms/ui/themes/default/html/views/folder/advanced.php
@@ -15,40 +15,33 @@
<table width="<?php echo O::escapeHtml('100%') ?>" class="<?php echo O::escapeHtml('or-table') ?>"><?php echo O::escapeHtml('') ?>
<tr class="<?php echo O::escapeHtml('or-headline') ?>"><?php echo O::escapeHtml('') ?>
<td class="<?php echo O::escapeHtml('or-help') ?>"><?php echo O::escapeHtml('') ?>
- <input type="<?php echo O::escapeHtml('checkbox') ?>" name="<?php echo O::escapeHtml('checkall') ?>" value="<?php echo O::escapeHtml('1') ?>" <?php if(@$checkall){ ?>checked="<?php echo O::escapeHtml('checked') ?>"<?php } ?> class="<?php echo O::escapeHtml('or-form-checkbox') ?>" /><?php echo O::escapeHtml('') ?>
- </td>
- <td class="<?php echo O::escapeHtml('or-help') ?>"><?php echo O::escapeHtml('') ?>
<span><?php echo O::escapeHtml(''.@O::lang('TYPE').'') ?>
</span>
- <span><?php echo O::escapeHtml(' / ') ?>
- </span>
+ </td>
+ <td class="<?php echo O::escapeHtml('or-help') ?>"><?php echo O::escapeHtml('') ?>
<span><?php echo O::escapeHtml(''.@O::lang('NAME').'') ?>
</span>
</td>
</tr>
<?php foreach((array)$object as $list_key=>$list_value) { extract($list_value); ?>
<tr class="<?php echo O::escapeHtml('or-data') ?>"><?php echo O::escapeHtml('') ?>
- <td width="<?php echo O::escapeHtml('1%') ?>"><?php echo O::escapeHtml('') ?>
+ <td><?php echo O::escapeHtml('') ?>
+ <i class="<?php echo O::escapeHtml('or-image-icon or-image-icon--action-'.@$icon.'') ?>"><?php echo O::escapeHtml('') ?>
+ </i>
+ </td>
+ <td><?php echo O::escapeHtml('') ?>
<?php $if7=($writable); if($if7) { ?>
- <input type="<?php echo O::escapeHtml('checkbox') ?>" name="<?php echo O::escapeHtml(''.@$id.'') ?>" value="<?php echo O::escapeHtml('1') ?>" <?php if(@$$id){ ?>checked="<?php echo O::escapeHtml('checked') ?>"<?php } ?> class="<?php echo O::escapeHtml('or-form-checkbox') ?>" /><?php echo O::escapeHtml('') ?>
+ <label><?php echo O::escapeHtml('') ?>
+ <input type="<?php echo O::escapeHtml('checkbox') ?>" name="<?php echo O::escapeHtml(''.@$id.'') ?>" value="<?php echo O::escapeHtml('1') ?>" <?php if(@$$id){ ?>checked="<?php echo O::escapeHtml('checked') ?>"<?php } ?> class="<?php echo O::escapeHtml('or-form-checkbox') ?>" /><?php echo O::escapeHtml('') ?>
+ <span class="<?php echo O::escapeHtml('or-form-label') ?>"><?php echo O::escapeHtml(''.@$name.'') ?>
+ </span>
+ </label>
<?php } ?>
- <?php $if7=(!writable); if($if7) { ?>
- <span><?php echo O::escapeHtml(' ') ?>
+ <?php if(!$if7) { ?>
+ <span><?php echo O::escapeHtml(''.@$name.'') ?>
</span>
<?php } ?>
</td>
- <td class="<?php echo O::escapeHtml('or-act-clickable') ?>"><?php echo O::escapeHtml('') ?>
- <label class="<?php echo O::escapeHtml('or-label') ?>"><?php echo O::escapeHtml('') ?>
- <a target="<?php echo O::escapeHtml('_self') ?>" date-name="<?php echo O::escapeHtml(''.@$name.'') ?>" name="<?php echo O::escapeHtml(''.@$name.'') ?>" data-type="<?php echo O::escapeHtml('open') ?>" data-action="<?php echo O::escapeHtml(''.@$type.'') ?>" data-method="<?php echo O::escapeHtml('') ?>" data-id="<?php echo O::escapeHtml(''.@$objectid.'') ?>" data-extra="<?php echo O::escapeHtml('[]') ?>" href="<?php echo O::escapeHtml('#/'.@$type.'/'.@$objectid.'') ?>" class="<?php echo O::escapeHtml('or-link') ?>"><?php echo O::escapeHtml('') ?>
- <i class="<?php echo O::escapeHtml('or-image-icon or-image-icon--action-'.@$icon.'') ?>"><?php echo O::escapeHtml('') ?>
- </i>
- <span><?php echo O::escapeHtml(''.@$name.'') ?>
- </span>
- <span><?php echo O::escapeHtml(' ') ?>
- </span>
- </a>
- </label>
- </td>
</tr>
<?php } ?>
<?php $if4=(($object)==FALSE); if($if4) { ?>
@@ -59,20 +52,6 @@
</td>
</tr>
<?php } ?>
- <tr class="<?php echo O::escapeHtml('or-data') ?>"><?php echo O::escapeHtml('') ?>
- <td><?php echo O::escapeHtml('') ?>
- <span><?php echo O::escapeHtml(' ') ?>
- </span>
- </td>
- <td colspan="<?php echo O::escapeHtml('2') ?>" class="<?php echo O::escapeHtml('or-act-clickable') ?>"><?php echo O::escapeHtml('') ?>
- <a target="<?php echo O::escapeHtml('_self') ?>" data-type="<?php echo O::escapeHtml('dialog') ?>" data-action="<?php echo O::escapeHtml('folder') ?>" data-method="<?php echo O::escapeHtml('create') ?>" data-id="<?php echo O::escapeHtml('') ?>" data-extra-dialogAction="<?php echo O::escapeHtml('folder') ?>" data-extra-dialogMethod="<?php echo O::escapeHtml('create') ?>" data-extra="<?php echo O::escapeHtml('{\'dialogAction\':\'folder\',\'dialogMethod\':\'create\'}') ?>" href="<?php echo O::escapeHtml('#/folder') ?>" class="<?php echo O::escapeHtml('or-link') ?>"><?php echo O::escapeHtml('') ?>
- <i class="<?php echo O::escapeHtml('or-image-icon or-image-icon--method-add') ?>"><?php echo O::escapeHtml('') ?>
- </i>
- <span><?php echo O::escapeHtml(''.@O::lang('menu_folder_create').'') ?>
- </span>
- </a>
- </td>
- </tr>
</table>
</div>
</div>
@@ -93,17 +72,11 @@
<h3 class="<?php echo O::escapeHtml('or-fieldset-label') ?>"><?php echo O::escapeHtml('') ?>
</h3>
<div class="<?php echo O::escapeHtml('or-fieldset-value') ?>"><?php echo O::escapeHtml('') ?>
- <div class="<?php echo O::escapeHtml('or-label') ?>"><?php echo O::escapeHtml('') ?>
- </div>
- <div class="<?php echo O::escapeHtml('or-value') ?>"><?php echo O::escapeHtml('') ?>
+ <label><?php echo O::escapeHtml('') ?>
<input type="<?php echo O::escapeHtml('radio') ?>" name="<?php echo O::escapeHtml('type') ?>" value="<?php echo O::escapeHtml(''.@$actiontype.'') ?>" <?php if(@$type=='${actiontype}'){ ?>checked="<?php echo O::escapeHtml('checked') ?>"<?php } ?> class="<?php echo O::escapeHtml('or-form-radio') ?>" /><?php echo O::escapeHtml('') ?>
- <label class="<?php echo O::escapeHtml('or-label') ?>"><?php echo O::escapeHtml('') ?>
- <span><?php echo O::escapeHtml(' ') ?>
- </span>
- <span><?php echo O::escapeHtml(''.@O::lang('FOLDER_SELECT_'.@$actiontype.'').'') ?>
- </span>
- </label>
- </div>
+ <span class="<?php echo O::escapeHtml('or-form-label') ?>"><?php echo O::escapeHtml(''.@O::lang('FOLDER_SELECT_'.@$actiontype.'').'') ?>
+ </span>
+ </label>
</div>
</section>
<?php } ?>
@@ -111,35 +84,23 @@
<h3 class="<?php echo O::escapeHtml('or-fieldset-label') ?>"><?php echo O::escapeHtml('') ?>
</h3>
<div class="<?php echo O::escapeHtml('or-fieldset-value') ?>"><?php echo O::escapeHtml('') ?>
- <div class="<?php echo O::escapeHtml('or-label') ?>"><?php echo O::escapeHtml('') ?>
- </div>
- <div class="<?php echo O::escapeHtml('or-value') ?>"><?php echo O::escapeHtml('') ?>
- <span><?php echo O::escapeHtml(' ') ?>
+ <label><?php echo O::escapeHtml('') ?>
+ <input type="<?php echo O::escapeHtml('checkbox') ?>" name="<?php echo O::escapeHtml('confirm') ?>" value="<?php echo O::escapeHtml('1') ?>" <?php if(@$confirm){ ?>checked="<?php echo O::escapeHtml('checked') ?>"<?php } ?> class="<?php echo O::escapeHtml('or-form-checkbox') ?>" /><?php echo O::escapeHtml('') ?>
+ <span class="<?php echo O::escapeHtml('or-form-label') ?>"><?php echo O::escapeHtml(''.@O::lang('CONFIRM_DELETE').'') ?>
</span>
- <input type="<?php echo O::escapeHtml('checkbox') ?>" name="<?php echo O::escapeHtml('confirm') ?>" value="<?php echo O::escapeHtml('1') ?>" <?php if(@$confirm){ ?>checked="<?php echo O::escapeHtml('checked') ?>"<?php } ?> required="<?php echo O::escapeHtml('required') ?>" class="<?php echo O::escapeHtml('or-form-checkbox') ?>" /><?php echo O::escapeHtml('') ?>
- <label class="<?php echo O::escapeHtml('or-label') ?>"><?php echo O::escapeHtml('') ?>
- <span><?php echo O::escapeHtml(''.@O::lang('CONFIRM_DELETE').'') ?>
- </span>
- </label>
- </div>
+ </label>
</div>
</section>
<section class="<?php echo O::escapeHtml('or-fieldset') ?>"><?php echo O::escapeHtml('') ?>
- <h3 class="<?php echo O::escapeHtml('or-fieldset-label') ?>"><?php echo O::escapeHtml('') ?>
+ <h3 class="<?php echo O::escapeHtml('or-fieldset-label') ?>"><?php echo O::escapeHtml(''.@O::lang('FOLDER_SELECT_TARGET_FOLDER').'') ?>
</h3>
<div class="<?php echo O::escapeHtml('or-fieldset-value') ?>"><?php echo O::escapeHtml('') ?>
- <div class="<?php echo O::escapeHtml('or-label') ?>"><?php echo O::escapeHtml('') ?>
- <span><?php echo O::escapeHtml(''.@O::lang('FOLDER_SELECT_TARGET_FOLDER').'') ?>
- </span>
- </div>
- <div class="<?php echo O::escapeHtml('or-value') ?>"><?php echo O::escapeHtml('') ?>
- <div class="<?php echo O::escapeHtml('or-selector') ?>"><?php echo O::escapeHtml('') ?>
- <input type="<?php echo O::escapeHtml('hidden') ?>" name="<?php echo O::escapeHtml('targetobjectid') ?>" value="<?php echo O::escapeHtml(''.@$targetobjectid.'') ?>" class="<?php echo O::escapeHtml('or-selector-link-value') ?>" /><?php echo O::escapeHtml('') ?>
- <input type="<?php echo O::escapeHtml('text') ?>" name="<?php echo O::escapeHtml('targetobjectid_text') ?>" placeholder="<?php echo O::escapeHtml(''.@$rootfoldername.'') ?>" value="<?php echo O::escapeHtml(''.@$rootfoldername.'') ?>" class="<?php echo O::escapeHtml('or-selector-link-name') ?>" /><?php echo O::escapeHtml('') ?>
- <div class="<?php echo O::escapeHtml('or-dropdown or-act-selector-search-results') ?>"><?php echo O::escapeHtml('') ?>
- </div>
- <div type="<?php echo O::escapeHtml('hidden') ?>" data-types="<?php echo O::escapeHtml('folder') ?>" data-init-id="<?php echo O::escapeHtml(''.@$rootfolderid.'') ?>" data-init-folderid="<?php echo O::escapeHtml(''.@$rootfolderid.'') ?>" class="<?php echo O::escapeHtml('or-navtree or-act-load-selector-tree') ?>"><?php echo O::escapeHtml('') ?>
- </div>
+ <div class="<?php echo O::escapeHtml('or-selector') ?>"><?php echo O::escapeHtml('') ?>
+ <input type="<?php echo O::escapeHtml('hidden') ?>" name="<?php echo O::escapeHtml('targetobjectid') ?>" value="<?php echo O::escapeHtml(''.@$targetobjectid.'') ?>" class="<?php echo O::escapeHtml('or-selector-link-value') ?>" /><?php echo O::escapeHtml('') ?>
+ <input type="<?php echo O::escapeHtml('text') ?>" name="<?php echo O::escapeHtml('targetobjectid_text') ?>" placeholder="<?php echo O::escapeHtml(''.@$rootfoldername.'') ?>" value="<?php echo O::escapeHtml(''.@$rootfoldername.'') ?>" class="<?php echo O::escapeHtml('or-selector-link-name') ?>" /><?php echo O::escapeHtml('') ?>
+ <div class="<?php echo O::escapeHtml('or-dropdown or-act-selector-search-results') ?>"><?php echo O::escapeHtml('') ?>
+ </div>
+ <div type="<?php echo O::escapeHtml('hidden') ?>" data-types="<?php echo O::escapeHtml('folder') ?>" data-init-id="<?php echo O::escapeHtml(''.@$rootfolderid.'') ?>" data-init-folderid="<?php echo O::escapeHtml(''.@$rootfolderid.'') ?>" class="<?php echo O::escapeHtml('or-navtree or-act-load-selector-tree') ?>"><?php echo O::escapeHtml('') ?>
</div>
</div>
</div>
diff --git a/modules/cms/ui/themes/default/html/views/folder/advanced.tpl.src.xml b/modules/cms/ui/themes/default/html/views/folder/advanced.tpl.src.xml
@@ -1,94 +1,57 @@
-<output xmlns="http://www.openrat.de/template" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.openrat.de/template ../../../../../../../template_engine/components/template.xsd">
- <form action="folder" subaction="edit">
- <table>
- <row class="headline">
- <column class="help">
- <!--
- <text value="${message:SELECT}"></text>
- -->
- <checkbox name="checkall"/>
- </column>
- <column class="help">
- <text value="${message:TYPE}"/>
- <text value=" / "/>
- <text value="${message:NAME}"/>
- </column>
- </row>
- <list list="${object}" extract="true">
- <row class="data">
- <column width="1%">
- <if true="${writable}">
- <checkbox name="${id}"/>
- </if>
- <if false="writable">
- <text value=" "/>
- </if>
- </column>
- <column class="act-clickable">
- <label for="${id}">
- <link type="open" name="${name}" action="${type}" id="${objectid}">
- <image action="${icon}"/>
- <text value="${name}" maxlength="40"/>
- <text value=" "/>
- </link>
- </label>
- </column>
- </row>
- </list>
- <if empty="${object}">
- <row>
- <column colspan="2">
- <text value="${message:NOT_FOUND}"/>
- </column>
- </row>
- </if>
- <row class="data">
- <column>
- <text value=" "/>
- </column>
- <column colspan="2" class="act-clickable">
- <link type="dialog" action="folder" subaction="create">
- <image method="add"/>
- <text value="${message:menu_folder_create}"/>
- </link>
- </column>
- </row>
- </table>
- <group title="${message:options}">
- <set var="type" value="${defaulttype}"/>
- <list list="${actionlist}" value="actiontype">
- <fieldset class="line" label="">
- <part class="label">
- </part>
- <part class="value">
- <radio name="type" value="${actiontype}"/>
- <label for="type" value="${actiontype}">
- <text value=" "/>
- <text value="${message:FOLDER_SELECT_${actiontype}}"/>
- </label>
- </part>
- </fieldset>
- </list>
- <fieldset class="line" label="">
- <part class="label">
- </part>
- <part class="value">
- <text value=" "/>
- <checkbox name="confirm" required="true"/>
- <label for="confirm">
- <text value="${message:CONFIRM_DELETE}"/>
- </label>
- </part>
- </fieldset>
- <fieldset class="line" label="">
- <part class="label">
- <text value="${message:FOLDER_SELECT_TARGET_FOLDER}"/>
- </part>
- <part class="value">
- <selector types="folder" param="targetobjectid" name="${rootfoldername}" id="${rootfolderid}" folderid="${rootfolderid}"/>
- </part>
- </fieldset>
- </group>
- </form>
- <!-- <insert script="mark" inline="true"></insert> -->
+<output xmlns="http://www.openrat.de/template" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+ xsi:schemaLocation="http://www.openrat.de/template ../../../../../../../template_engine/components/template.xsd">
+ <form action="folder" subaction="edit">
+ <table>
+ <row class="headline">
+ <column class="help">
+ <!--
+ <text value="${message:SELECT}"></text>
+ -->
+ <text value="${message:TYPE}"/>
+ <!-- <checkbox name="checkall" /> -->
+ </column>
+ <column class="help">
+ <!-- <text value=" / "/> -->
+ <text value="${message:NAME}"/>
+ </column>
+ </row>
+ <list list="${object}" extract="true">
+ <row class="data">
+ <column>
+ <image action="${icon}"/>
+ </column>
+ <column>
+ <if true="${writable}">
+ <checkbox name="${id}" label="${name}"/>
+ </if>
+ <else>
+ <text value="${name}"/>
+ </else>
+ </column>
+ </row>
+ </list>
+ <if empty="${object}">
+ <row>
+ <column colspan="2">
+ <text value="${message:NOT_FOUND}"/>
+ </column>
+ </row>
+ </if>
+ </table>
+ <group title="${message:options}">
+ <set var="type" value="${defaulttype}"/>
+ <list list="${actionlist}" value="actiontype">
+ <fieldset label="">
+ <radio name="type" value="${actiontype}" label="${message:FOLDER_SELECT_${actiontype}}"/>
+ </fieldset>
+ </list>
+ <fieldset label="">
+ <checkbox name="confirm" label="${message:CONFIRM_DELETE}"/>
+ </fieldset>
+ <fieldset label="${message:FOLDER_SELECT_TARGET_FOLDER}">
+ <selector types="folder" param="targetobjectid" name="${rootfoldername}" id="${rootfolderid}"
+ folderid="${rootfolderid}"/>
+ </fieldset>
+ </group>
+ </form>
</output>
diff --git a/modules/template_engine/components/html/component_user/component-user.php b/modules/template_engine/components/html/component_user/component-user.php
@@ -1,16 +1,34 @@
<?php
-function component_user( $user )
+
+use cms\base\Configuration as C;
+use cms\base\Language;
+use language\Messages;
+
+function component_user($user )
{
extract( $user );
-
- if ( empty($name) )
- $name = \cms\base\Language::lang('UNKNOWN');
+
if ( empty($fullname) )
- $fullname = \cms\base\Language::lang('NO_DESCRIPTION_AVAILABLE');
+ $fullname = Language::lang( Messages::UNKNOWN );
- if ( isset($mail) && $mail && \cms\base\Configuration::subset(['security','user'])->is('show_mail',false ) )
- echo "<a href=\"mailto:$mail\" title=\"$fullname\">$name</a>";
+ $showName = C::subset(['security','user'])->is('show_username',false );
+
+ if ( $showName ) {
+ if ( empty($name) )
+ $name = Language::lang(Messages::UNKNOWN );
+ }
else
- echo "<span title=\"$fullname\">$name</span>";
-}
-?>-
\ No newline at end of file
+ $name = $fullname;
+
+
+ $showMail = isset($mail) && $mail && C::subset(['security','user'])->is('show_mail',false );
+
+ if ( $showMail )
+ echo "<a href=\"mailto:$mail\" title=\"$fullname\">";
+
+ echo "<span class=\"or-username\" title=\"$fullname\">$name</span>";
+
+ if ( $showMail )
+ echo "</a>";
+
+}+
\ No newline at end of file