Hi rvelices and mistic100,
In Piwigo 2.5 we have a new behavior related to $page['section_title'].
The problem is on a page like picture.php?/1234 with no section specified. The "Home" link goes to /index.php?/categories/flat instead of /.
I don't think this behavior is expected (or please explain). Can you confirm? Can you fix it?
Offline
Related commits : [Subversion] r18637 and [Subversion] r19233
Offline
function params_for_duplication (called in duplicate_index_url) adds flat=>true in $params for make_index_url.
Offline
The problem also exists when you go on /picture.php?/1234/category (which happens from PWG Stuffs random photos block, or from Advanced Menu Manager Random Picture)
Offline
plg wrote:
The problem is on a page like picture.php?/1234 with no section specified. The "Home" link goes to /index.php?/categories/flat instead of /.
I don't think this behavior is expected (or please explain). Can you confirm? Can you fix it?
I agree, it is not good (I'll fix it when i have time)
line 581 section_init
$used_link = duplicate_index_url(array('start'=>0));
should be:
$used_link = duplicate_index_url(array('start'=>0), array('flat'));
In my initial thoughts, I wanted on the picture page to have the breadcrumb navigate to navigate up as the UP icon (I realized that on picture page most people never click the up icon or the photo, but they expect to fall on the previous page)
I'll try to make it work in 2.6 ...
Offline
Hi rvelices,
Thank you for [Subversion] r22161, but I still have /index.php?/categories instead of a simple / (which is not the same when you have an home page with plugin Additionnal Pages)
Offline
plg wrote:
Hi rvelices,
Thank you for [Subversion] r22161, but I still have /index.php?/categories instead of a simple / (which is not the same when you have an home page with plugin Additionnal Pages)
Ok. Can you try and see if it fits with your configs :
if (isset($page['title'])) { $page['section_title'] = '<a href="'.get_gallery_home_url().'">'.l10n('Home').'</a>'; if (!empty($page['title'])) { $page['section_title'] .= $conf['level_separator'].$page['title']; } else { $page['title'] = $page['section_title']; } }
Offline
Yes, it works much better with this code in include/section_init.inc.php
Offline
[Bugtracker] ticket 2891
I have commited the change you proposed (and that is working fine for me) in [Subversion] r22282
Offline