I realize this is set in shadowbox.js which is separate from the drupal module but perhaps there could be clarification on the admin/settings page to add a trailing "/" or maybe the module could add it itself.

Totally not a big deal to me but I wanted to bring it to attention for ya ;)

Example drupal error log:

Type: page not found
Location: http://example.com/sites/all/plugins/shadowbox-build-3.0blibraries/swfob...

There should be a "/" after "shadowbox-build-3.0b"

I put the "/" in my shadowbox.js and the error was gone. (S.path+"/libraries/swfobject/swfobject.js")

I also saw that there was a ticket closed on this already for the dev version but this is on the stable version so I opened a new one.

Comments

dstol’s picture

dstol
Primary language English
commented

You can resolve this by adding a trailing slash to Shadowbox location in admin/settings/shadowbox/global

onejam’s picture

onejam commented

I'm getting the same error shown in my log file:

sites/all/libraries/shadowboxlibraries/swfobject/swfobject.js

In Shadow location, i have: sites/all/libraries/shadowbox

As you can see, it's no adding the slash after it.

Is this fixed in the dev version?

Thanks,

psynaptic’s picture

psynaptic commented
Status: Active » Fixed

Thanks for the work guys.

Fixed in DRUPAL-6--3.

onejam’s picture

onejam commented

Thanks, I can confirm it fixed the issue.

psynaptic’s picture

psynaptic commented

Thanks for confirming. I'll fix a couple more issues and make a new release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

psynaptic’s picture

psynaptic commented
Status: Closed (fixed) » Needs review

I have just committed a fix for this issue. Please could you test it and report if the problem is solved for you?

strikehawkecomm’s picture

strikehawkecomm commented

Add to shadowbox module the eval that if the trailing slash is missing, ADD it?

psynaptic’s picture

psynaptic commented

WTF?

strikehawkecomm’s picture

strikehawkecomm commented

Changed comment to a question to clarify it wasn't meant to be a demand.

psynaptic’s picture

psynaptic commented
Status: Needs review » Fixed

I have no idea what you were trying to say. Could you clarify?

I made a new release anyway.

Status: Fixed » Closed (fixed)
Issue tags: -shadowbox.js

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +shadowbox.js

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.