Piwigo / piwigo-menutree

Dynamic album menu tree for piwigo (plugin)
0 stars 2 forks source link

Do not show hidden categories in rv-menutree #1

Closed BenMak closed 7 years ago

BenMak commented 7 years ago

Currently RV-menutree also shows the categories which have in the namefield \<!--mb-hidden-->. To my opinion this is not correct. I suggest to add in rv_menutree_categories.tpl after {$cat.PRE} on line 7: {if !$cat.NAME|stristr:''} and before {/foreach} on line 15: {/if}

_{foreach from=$block->data.MENU_CATEGORIES item=cat} {$cat.PRE} {if !$cat.NAME|stristr:'\\'} <li{$cat.CLASS}> <a href="{$cat.URL}"{if $cat.IS_UPPERCAT} rel="up"{/if}>{$cat.NAME} {if $cat.count_images > 0} \<span{if $cat.nb_images <= 0} class=menuInfoCatByChild{/if} title="{$cat.TITLE}"> [{$cat.count_images}] \</span> {/if} {if !empty($cat.icon_ts)} \<img title="{$cat.icon_ts.TITLE}" src="{$ROOT_URL}{$themeconf.icon_dir}/recent{if $cat.icon_ts.IS_CHILD_DATE}_bychild{/if}.png" alt="(!)"> {/if} {/if} {/foreach}

modus75 commented 7 years ago

That feature is part of another plug-in (not core). Your solution does not work if you hide an album that has children which are not hidden.

However I will look into how to make this plug-in compatible with extended description plugin.

On Jan 19, 2017 2:45 PM, "BenMak" notifications@github.com wrote:

Currently RV-menutree also shows the categories which have in the namefield . To my opinion this is not correct. I suggest to add in rv_menutree_categories.tpl after {$cat.PRE} on line 7: {if !$cat.NAME|stristr:''} and before {/foreach} on line 15: {/if}

{foreach from=$block->data.MENU_CATEGORIES item=cat} {$cat.PRE} {if !$cat.NAME|stristr:''} <li{$cat.CLASS}> <a href="{$cat.URL}"{if $cat.IS_UPPERCAT} rel="up"{/if}>{$cat.NAME} {if $cat.count_images > 0} <span{if $cat.nb_images <= 0} class=menuInfoCatByChild{/if} title="{$cat.TITLE}"> [{$cat.count_images}] {/if} {if !empty($cat.icon_ts)} [image: (!)] {/if} {/if} {/foreach}

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/modus75/piwigo-menutree/issues/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AJRc9sfscbTeCnRIMcheNWcwzCJOevzoks5rT2jvgaJpZM4LoH_d .

BenMak commented 7 years ago

modus75,

Thanks for your fast action. Works well for me!

Regards, Ben Makkink

2017-03-14 21:33 GMT+01:00 modus75 notifications@github.com:

Closed #1 https://github.com/modus75/piwigo-menutree/issues/1 via 5c44e8d https://github.com/modus75/piwigo-menutree/commit/5c44e8dde9a563f153214f628cf4682b2fcd4e94 .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/modus75/piwigo-menutree/issues/1#event-999787429, or mute the thread https://github.com/notifications/unsubscribe-auth/ARdqtD50ktlMIwLPbgbL__DIpJoFQjkCks5rlvmogaJpZM4LoH_d .