Disclaimer doesn't show up

Well unfortunately all I can add to this is a "me too!".

I'm having the exact same problem. Thought disclaimer would be a nice easy way to display a page that the user has to agree to to enter the site, but it's not working.

At first I thought it was related to colorbox. I worked on that for a bit and got colorbox working now (needs newer jQuery, use jQuery-update module), so I don't think that colorbox is the problem now since I can get colorbox to display on other parts of the site.

So maybe me posting on this will make it more active and someone will have a solution?

Or at least a spot where I should start looking / troubleshooting?

any suggestions appreciated!

thx

I know this has been open for a while but this functionality was a must have so I was digging around in the code and I think I found two distinct problems:

#1. nyroModal == broken, use jqModal instead
(I didn't investigate nyroModal any closer once I realized that jqModal worked)

#2. This module uses base_path(), which if you do not have a base path set is a problem because the URL that it attempts to load is then http://disclaimer/ instead of http://mysite.com/disclaimer

I've simply opted to hack out the reference to base_path from the module in the function disclaimer_get_js().

Sorry for the delay, in fact 7 release is not functionnal, i don't get time to finish port, i will try to publish a working release in next weeks.

Regards.
Mog.

I was able to fix the lack of content when using jqModal by including the $base_url into the ajax request, patch attached... Not sure if using the global is best practice, please review

*sigh* re-rolled that earlier patch to correct the quoting of the ajax path

Don't ask me how but I still had a quoting problem, this finally should be correct, sorry peeps

Thanks Steve! :) I've got another patch which moves the cookie check from PHP into JS which improves the module no end as it makes it support boost and caching properly. Will try and re-roll it next week and upload as I'm sure it'd be useful.

Couldn't say directly but generally speaking check the console for any JS errors. Are you using the age verification bit? I'm not using that so I've not tested it directly in D7.

The only changes I've made to the code which affect the JS functionality are to do with the cookie expiry, there's a deprecated JS function in use (toGMTString which has been replaced with toUTCString) but that is only triggered if you're setting a non-default disclaimer_cookie_expires variable as I am.

Whole swathes of this module could probably be re-written now as various bits are out of date with D7.

Attached is a patch against 7.x-1.x-dev which includes all the tweaks I've made to date if that's any use...

ghazlewood,
thank you! dev version + your patch looks like worked like a charm for me. How about releasing an official dev version for drupal 7?

Thank you again

Thanks for the patch, i finally get some time to release a real 7.x version of this module, please check the new dev release and give feedback.

I have installed the disclaimer and colorbox module and have configured them as well but still the disclaimer does not show up.

I tried patching it with the latest patch posted by you but it is giving me the following error

$ git apply -v disclaimer-7.x-1.x-dev-2012-10-15-001.patch
Checking patch disclaimer.js...
error: while searching for:
/**
* @name disclaimer module js file
*/
function WriteCookie(nom, valeur){
var cookie_path = Drupal.settings.disclaimer.cookie_path;
var cookie_expires = Drupal.settings.disclaimer.cookie_expires;
var cookie_domain = Drupal.settings.disclaimer.cookie_domain;

error: patch failed: disclaimer.js:1
error: disclaimer.js: patch does not apply
Checking patch disclaimer.module...
error: while searching for:
*/
function disclaimer_init() {
if (disclaimer_show()) {
// add modal library
drupal_add_library('disclaimer', variable_get('disclaimer_modal', 'nyroModal
'));
// cookie settings

error: patch failed: disclaimer.module:44
error: disclaimer.module: patch does not apply

Last dev version do not need this patch.