But,....

1. /Admin/Site Configuration/Brilliant Gallery/Configure Permissions/
. - directs me to my main drupal directory
. - Not any permisssions page
. - Is this supposed to take me to the User Permissions settings page?

2. /Admin/Site Configuration/Brilliant Gallery/Manage Galleries/
. - Open gallery list of pictures
. - Select Visibility
. - States 'saving...' and never changes.
. - I have tried different file sizes and nothing seems to commit.

I also am not able to define a path to the gallery to show images as you've indicated on http://vacilando.net/bg

Thanks
ORA

CommentFileSizeAuthor
#7 bg-379026.patch1.53 KBdrpchris
#5 bgchecklist.txt2.7 KBegon.ojamaa

Comments

danmc’s picture

danmc commented

The page at http://vacilando.net/bg is blank. Is there somewhere else to find install and use documentation?

I too am stuck getting anything setup with this module.

vm’s picture

vm commented

The documentation page on the links you provide work on this end.

orasca’s picture

orasca commented

The documentation page/link works for me. Possibly copy/paste to a browser. It can be viewed in FF, IE and GC (Google Chrome).

egon.ojamaa’s picture

egon.ojamaa commented
Priority: Normal » Major

. - States 'saving...' and never changes.
this bug is because your drupal is inside folder and basepath is wrong in this modules file: bgchecklist.js
Makes it unusable to very many sites.

I used "Drupal.settings.basePath" to fix it.
Fixed file included. Dont forget to rename it to bgchecklist.js.

As of this permissions page bug.. the link is broken.
Just go to persmissions page manually and brilliant gallery checkboxes are there.

egon.ojamaa’s picture

egon.ojamaa commented
StatusFileSize
new bgchecklist.txt2.7 KB

for some reasone attaching didnt work... so..

find: $.getJSON("/bgchecklist/loadall/"+nodeid,
replace: $.getJSON(Drupal.settings.basePath +"/bgchecklist/loadall/"+nodeid,

find: $.get("/bgchecklist/save/"+nodeid+"/"+$(this).attr("id")+"/0",
replace: $.get(Drupal.settings.basePath +"/bgchecklist/save/"+nodeid+"/"+$(this).attr("id")+"/0",

find: $.get("/bgchecklist/save/"+nodeid+"/"+$(this).attr("id")+"/1",
replace: $.get(Drupal.settings.basePath +"/bgchecklist/save/"+nodeid+"/"+$(this).attr("id")+"/1",

vm’s picture

vm commented
Version: 6.x-3.0 » 6.x-3.x-dev
Component: Miscellaneous » Code
Category: support » bug
Status: Active » Needs work

This should be a patch file so that it can be reviewed and rolled into the -dev

marking as needs work

drpchris’s picture

drpchris commented
Title: Brilliant Gallery seems like it would be a nice module to use » bgchecklist.js not using correct path
Status: Needs work » Patch (to be ported)
StatusFileSize
new bg-379026.patch1.53 KB

created a patch from the above code corrections

vm’s picture

vm commented
Status: Patch (to be ported) » Needs review
vacilando’s picture

vacilando commented
Version: 6.x-3.x-dev » 6.x-4.x-dev
Status: Needs review » Fixed

Thanks guys! Patch applied to 6.x-4.x-dev.

vacilando’s picture

vacilando commented

Nope; it actually has to be without the initial slash:
$.get(Drupal.settings.basePath +"bgchecklist/...
because if Drupal is installed in ther root then Drupal.settings.basePath contains "/".

Committed to dev (6 as well as 7).

Status: Fixed » Closed (fixed)

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