next »

[delicia]

i want to hide some events from visitors unless they are logged in. i set up an event field called hidden which is set to yes or no when event is entered or edited. i downloaded and installed the plug in for private events and edited as shown below:

# ADDED         # Skip if it's private
#         next unless ($event_details->{$s->{'event_id'}}->{'author_id'} eq $main::User->{'username'});
         next unless (($event_details->{$s->{'event_id'}}->{'hidden'} eq "no") | | ($Session));

it seems to work for me -- when i'm logged in i can see the hidden events, but when i'm not logged in i can't. however, i sent a link to my test calendar to a friend and she could see the events set to yes. i did not give her any login info.

could someone tell me what i'm doing wrong? thanks.

[DanO]

Even if a user hasn't logged in the $Session variable is still set when they view the calendar (providing their browser is accepting cookies). The difference however is if they're not logged in, they have a user name of "anonymous".

You'll have to make your test conditional upon that.

BTW. I don't think your changes with limit the search results.

Dan O.

[This message has been edited by DanO (edited December 17, 2003).]

[delicia]

i have my browser set to prompt for cookies but i'm not getting a prompt for the calendar. i still don't see 'hidden' events unless logged in. i'm going straight to default calendar without searching.

------------------

[DanO]

If you want to post a link to the calendar and tell us which events shouldn't be displaying, others can test it for you. If not, all I can suggest is you try my suggestion and ask your friend to try again.

Dan O.

[This message has been edited by DanO (edited December 17, 2003).]

[delicia]

here's the link: http://www.delicia.com/cgi-bin/cs3/calendar.pl

december 23 -- you should not see an event unless you are logged in.

------------------

[DanO]

I see it.  

Did you try the change I suggested?

Dan O.

------------------

[delicia]

i think so, but not sure of field names. this is what i did:

         next if (($event_details->{$s->{'event_id'}}->{'hidden'} eq "yes") && ($User->{name} eq 'anonymous'));

i also edited default.html to display the value of of those two fields for each event.

------------------

[This message has been edited by delicia (edited December 18, 2003).]

[DanO]

** not sure of field names **

It should be $User->{username} not $User->{name}

Try

next if (($event_details->{$s->{'event_id'}}->{'hidden'} eq "yes") && (!$User->{username} | | $User->{username} eq 'anonymous'));

(all on one line)

     NOTE: This forum software adds a space between double bars ( | | ) which would have to be removed from the code you use.

Dan O.

[This message has been edited by DanO (edited December 19, 2003).]

[delicia]

should username be enclosed in single quotes?

------------------
i guess not; i looked at other places in the script and it wasn't. i've implemented your suggestion and am awaiting my friend to test.

another question -- i noticed in the plugin, there is a usermessage that can be commented out to show that the script got to the plugin. it is never displayed (after i uncommented it); i looked in default.html and the message only appears under search, not under regular view -- correct?

and you are quite right that this plugin doesn't do what i want in a search. how would i customize the before_search part -- can i just set flags like:

if not logged in, hidden = no

or will i need to duplicate the before get event sub, or worse write a whole new routine (over my head)?

[This message has been edited by delicia (edited December 20, 2003).]

[This message has been edited by delicia (edited December 20, 2003).]

[DanO]

** should username be enclosed in single quotes? **

Technically yes, either single or double quotes but Perl is very forgiving.

** the message only appears under search, not under regular view -- correct? **

Yes... unless you modify the template to display it elsewhere.

** how would i customize the before_search part -- or will i need to duplicate the before get event sub **

You would need to modify the 'sub search' subroutine in the Event.inc file or duplicate it and add all the necessary code to the display_before_search.pl file along with a modified # SEARCH routine from the calendar.pl file (which calls 'sub search').

Or you could totally disable the search function by having the display_before_search.pl return a value of 1 instead of 0 (zero) like it originally is set to.

Dan O.

[This message has been edited by DanO (edited December 20, 2003).]