Am 6.10.2014 05:23, schrieb Markus:
> <f:section name="content">
>     <f:if condition="{news}">
> ...

Hallo Markus,

schon das Kompaktieren von <f:If><f:then> plus HTML-Tags macht es
übersichtlicher. Man merkt auch schnell, das dein zitiertes Template
überflüssigen Code enthält. (Ich habe ihn in der ersten Version nur
gekennzeichnet) und in der zweiten Version dann rausgeschmissen. (ich
habe beide nicht getestet.)
Wenn das zitierte Template aus der aktuellen tx_news-Version stammt,
könntest du nach einem Test einen Forge-Eintrag machen, um dem
Entwickler zu helfen, die Extension für alle besser zu machen.

<f:section name="content">
    <f:if condition="{news}"><f:then><div class="news-list-view">
        <f:if condition="{settings.templateLayout} == 2"><f:then>
            <f:for each="{news}" as="newsItem">
                <f:render partial="List/ItemEinsaetze" arguments="{newsItem: 
newsItem, settings:settings}"/>
            </f:for>
        </f:then><f:else>
            <f:if condition="{settings.hidePagination}"><f:then>
                <f:if condition="{settings.templateLayout} == 1"><f:then>
                    <f:for each="{news}" as="newsItem">
                        <f:render partial="List/ItemLatest" 
arguments="{newsItem: newsItem, settings:settings}"/>
                    </f:for>
                </f:then><f:else>
                    <f:for each="{news}" as="newsItem">
                        <f:render partial="List/Item" arguments="{newsItem: 
newsItem, settings:settings}"/>
                    </f:for>
                </f:else></f:if>
<f:comment>Überflüssig wegen übergeordneten If</f:comment>
                <f:if condition="{settings.templateLayout} == 2"><f:then>
                    <f:for each="{news}" as="newsItem">
                        <f:render partial="List/ItemEinsaetze"
                                  arguments="{newsItem: newsItem, 
settings:settings}"/>
                    </f:for>
                </f:then></f:if>
<f:comment>// Überflüssig wegen übergeordneten If</f:comment>
            
<f:comment>condition="settings.hidePagination"</f:comment></f:then><f:else>
                <n:widget.paginate objects="{news}" as="paginatedNews" 
configuration="{settings.list.paginate}" 
initial="{offset:settings.offset,limit:settings.limit}">
                    <f:if condition="{settings.templateLayout} == 1"><f:then>
                        <f:for each="{paginatedNews}" as="newsItem">
                            <f:render partial="List/ItemLatest" 
arguments="{newsItem: newsItem, settings:settings}"/>
                        </f:for>
                    </f:then><f:else>
                        <f:for each="{paginatedNews}" as="newsItem">
                            <f:render partial="List/Item" arguments="{newsItem: 
newsItem, settings:settings}"/>
                        </f:for>
                    </f:else></f:if>
<f:comment>Überflüssig wegen übergeordneten If</f:comment>
                    <f:if condition="{settings.templateLayout} == 2"><f:then>
                        <f:for each="{news}" as="newsItem">
                            <f:render partial="List/ItemEinsaetze" 
arguments="{newsItem: newsItem, settings:settings}"/>
                        </f:for>
                    </f:then></f:if>
<f:comment>Überflüssig wegen übergeordneten If</f:comment>
                </n:widget.paginate>
            </f:else></f:if>
        </f:else></f:if></div>
    <f:comment>condition="news"</f:comment></f:then><f:else><div 
class="no-news-found">
        <f:translate key="list_nonewsfound" />
    </div></f:else></f:if>
</f:section>

#########################################################
oder bereinigt passt es schon auf eine Bildschirmseite, 
was Übersicht schafft
#########################################################

<f:section name="content">
    <f:if condition="{news}"><f:then><div class="news-list-view">
        <f:if condition="{settings.templateLayout} == 2"><f:then>
            <f:for each="{news}" as="newsItem">
                <f:render partial="List/ItemEinsaetze" arguments="{newsItem: 
newsItem, settings:settings}"/>
            </f:for>
        </f:then><f:else>
            <f:if condition="{settings.hidePagination}"><f:then>
                <f:if condition="{settings.templateLayout} == 1"><f:then>
                    <f:for each="{news}" as="newsItem">
                        <f:render partial="List/ItemLatest" 
arguments="{newsItem: newsItem, settings:settings}"/>
                    </f:for>
                </f:then><f:else>
                    <f:for each="{news}" as="newsItem">
                        <f:render partial="List/Item" arguments="{newsItem: 
newsItem, settings:settings}"/>
                    </f:for>
                </f:else></f:if>
            
<f:comment>condition="settings.hidePagination"</f:comment></f:then><f:else>
                <n:widget.paginate objects="{news}" as="paginatedNews" 
configuration="{settings.list.paginate}" 
initial="{offset:settings.offset,limit:settings.limit}">
                    <f:if condition="{settings.templateLayout} == 1"><f:then>
                        <f:for each="{paginatedNews}" as="newsItem">
                            <f:render partial="List/ItemLatest" 
arguments="{newsItem: newsItem, settings:settings}"/>
                        </f:for>
                    </f:then><f:else>
                        <f:for each="{paginatedNews}" as="newsItem">
                            <f:render partial="List/Item" arguments="{newsItem: 
newsItem, settings:settings}"/>
                        </f:for>
                    </f:else></f:if>
                </n:widget.paginate>
            </f:else></f:if>
        </f:else></f:if>
    <f:comment>condition="news"</f:comment></div></f:then><f:else><div 
class="no-news-found">
        <f:translate key="list_nonewsfound" />
    </div></f:else></f:if>
</f:section>


Dieter

-- 
Dr. Dieter Porth - 
Mein kleines TYPO3-Labor: http://www.mobger.de/

_______________________________________________
TYPO3-german mailing list
TYPO3-german@lists.typo3.org
http://lists.typo3.org/cgi-bin/mailman/listinfo/typo3-german

Antwort per Email an