A+ | A - | RESET
Fixed | Wide | Full | Reset

Capture360.Net

Cover Every Angle

  Advanced search
Welcome, Guest. Please login or register.
Did you miss your activation email?
22:10 GMT - Fri 19 Apr 24

Login with username, password and session length
Pages: [1]
Author Topic: Drupal Breadcrumbs  (Read 14150 times)
0 Members and Guests are viewing this topic.
mh
Convert
******
Posts: 11


« at: 19:45 GMT - Wed 02 Jan 13 »

Interesting behavior under Drupal.   Gallery 2 has its own breadcrumbs which work well.

However, disabling the Drupal breadcrumb is mixed.   You can disable g2drupal from updating it by disabling the URL Rewrite plugin under Gallery 2.   But using something like the "disable_breadcrumbs" module in Drupal to eliminate it entirely does not work for some reason.   With the core modules it works okay, as with phpbbforum (for example) but not g2drupal.

There is also an issue under URL Rewrite where there is a missing leading slash ("/") in breadcrumb URLs on albums that have spaces (and other characters?) in their names.   For example:

Album name:  "Test Album"

Breadcrumb contains URL:  ". . . gallery/main. php/vTest&20Album"
Breadcrumb SHOULD read: ". . . gallery/main. php/v/Test+Album"

Though adding just the missing slash works fine regardless of space (%20) or plus (+) in the URL.
Logged
Dayo
Sheriff
*
Posts: 447


WWW
« Reply #1 at: 21:56 GMT - Wed 02 Jan 13 »

Disagree

G2 provides two user entries for albums

Name. Users should avoid spaces here if they do not want unslightly urls as it controls the url.
Title. This is just for display in the gallery and may contain anything.

G2 could easily have been set to just use one of these two for both reasons. Once a user chooses to put spaces in the album name, then they will get what they get.

Conclusion? Not a bug. Wont Fix.

Keep them coming though.


EDIT: Looked at your urls again. I got thrown by the change in "Test&20Album" to "Test+Album" and missed the slash missing on the first.

So breakdown is this:

On the change of "&20" to "+", wont fix
On the missing "/", will investigate.
« Last Edit: 22:32 GMT - Wed 02 Jan 13 by Dayo » Logged
Dayo
Sheriff
*
Posts: 447


WWW
« Reply #2 at: 22:03 GMT - Wed 02 Jan 13 »

On disabling the drupal breadcrumb.

There is an existing option to disable the g2 breadcrumb when within drupal even if ordinarily active in G2 but not for the drupal breabcrumb mainly because I wanted one in drupal.

I'll consider adding an option to do this as I can see that some might not want it.

Conclusion? Accepted.

Logged
mh
Convert
******
Posts: 11


« Reply #3 at: 22:36 GMT - Wed 02 Jan 13 »

Makes sense!  Good stuff.

. . . and awesome, much-needed module!
Logged
mh
Convert
******
Posts: 11


« Reply #4 at: 22:39 GMT - Wed 02 Jan 13 »

Forgot to mention - User Albums can generate albums with spaces because the user name has a space in it. . .
Logged
Dayo
Sheriff
*
Posts: 447


WWW
« Reply #5 at: 23:08 GMT - Wed 02 Jan 13 »

OK. Done some tests.

Actually found that the output URl has a "+" instead of "%20" and not the other way round. Conclusion? "Accepted". Will make sure the breadcrumb url has "%20" for spaces in place of "+". Now I am sure someone will pop up later and say they have an error because their paths include a deliberate "+". Best bet is to avoid spaces in album names though and if the user album module does create them as you say, then that should be patched accordingly.

On the missing "/". I can't reproduce this and looking at the code, cannot see how it can happen. Conclusion? "Works for me. Wont fix".
Logged
Dayo
Sheriff
*
Posts: 447


WWW
« Reply #6 at: 00:10 GMT - Thu 03 Jan 13 »

Further testing confirms my original thoughts that having spaces in album names will always lead to problems in G2 irrespective of whether this module is in use or not.

In summary, you cannot have spaces in your album names in G2. Status changed back to "Not a bug, wont fix".

So of the items discussed here so far, the only one going forward is the option to disable the Drupal breadcrumb and this has been implemented for the next version from the current 0.9.5 to be released soon.
Logged
Dayo
Sheriff
*
Posts: 447


WWW
« Reply #7 at: 19:43 GMT - Thu 03 Jan 13 »

I have looked further into the issue with items with spaces in the item names and it is a wider issue affecting the Gallery2 rewrite module.

A fix has been posted under the troubleshooting section of the G2Drupal Codex Page.
Logged
Pages: [1]
Jump to:  

Latest Posts
by Dayo
[21:04 GMT - Tue 19 Aug 14]

[22:20 GMT - Sat 16 Aug 14]

[21:15 GMT - Wed 09 Apr 14]

by Dayo
[17:18 GMT - Tue 21 Jan 14]

[15:55 GMT - Sun 15 Sep 13]
Powered by SMF 1.1.21 | SMF © 2015, Simple Machines
Corto design TinyPortal v0.9.8 © Bloc
Configured by Dayo Akanji
Contact