next »

[snooty1972]

Could someone take a look at my template and let me know where I'm going wrong. I'm also trying to limit the number of events show e.g. to the first 5 but so far the below is not working (this was based on a previous forum post)

Thanks

<%
&getEvents( {'duration'=>$in{duration}} );
$DAY = $Grid->{'grid'}->

• ->
• ;
  $EVENTS = $DAY->{events};
  $total_events=5;
  %>
  <%EVENTLOOP:%>
  <%FOREACH EVENTLIST%>
  <%IF EVENTS EXIST%>
  <BR>
  <%FOREACH EVENT%>
  <%$total_events++;%>
  <%if (total_events == 5) { last EVENTLOOP; } %>
  <img src="http://www.regenthall.co.uk/regent/images/plus.gif" width=15 height=9 border=0>
  <A class=title HREF="<%=$CGI_URL_QUERYSTRING%>calendar=&template=&view=Event&event_id=<%=$EVENT->{'details'}->{'id'}%>"><%= $EVENT->{'details'}->{'title'} %></A><BR>
  <% if ($EVENT->{'schedule'}->{'start_time'}) { %><%= $EVENT->{'schedule'}->{'start_time'} %><% } %>

  <% if ($EVENT->{'schedule'}->{'start_time'}) { %> : <% } %>
  <%=$DAY->{dayname}%>, <%=$DAY->{monthname}%> <%=$DAY->{dd}%><BR>
  <%/FOREACH%>
  <%/IF%>
  <%/FOREACH%>

  <%unless($total_events){%>
  <CENTER><I>No Events</I>
  <% } %>
  

[delicia]

on the first line where you set total_events = 5; change the variable name to $max_events. then when you do the comparison, change 5 to $max_events.

i think your problem is that you're setting total_events to 5 and then you increment it from there. so after the first event, your variable is 6. when you do the comparison, it's never exactly equal to 5.

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

[DanO]

<%if (total_events == 5) {

"total_events" is a string, it will never equal 5.

Dan O.

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

[snooty1972]

If I understood you correctly the new code would look as follows however this returns an error in line 12. Sorry, as you can see I am not a Pearl programmer. I'm guessing I've gotten the syntax wrong?

<%
&getEvents( {'duration'=>$in{duration}} );
$DAY = $Grid->{'grid'}->

• ->
• ;
  $EVENTS = $DAY->{events};
  $max_events=5;
  %>
  <%EVENTLOOP:%>
  <%FOREACH EVENTLIST%>
     <%IF EVENTS EXIST%>
     <BR>
        <%FOREACH EVENT%>
           <%$total_events++;%>
           <%if (total_events == max_events) { last EVENTLOOP; } %>
           <img src="http://www.regenthall.co.uk/regent/images/plus.gif" width=15 height=9 border=0>
           <A class=title HREF="<%=$CGI_URL_QUERYSTRING%>calendar=&template=&view=Event&event_id=<%=$EVENT->{'details'}->{'id'}%>"><%= $EVENT->{'details'}->{'title'} %></A><BR>
           <% if ($EVENT->{'schedule'}->{'start_time'}) { %><%= $EVENT->{'schedule'}->{'start_time'} %><% } %>

           <% if ($EVENT->{'schedule'}->{'start_time'}) { %> : <% } %>
           <%=$DAY->{dayname}%>, <%=$DAY->{monthname}%> <%=$DAY->{dd}%><BR>
        <%/FOREACH%>
     <%/IF%>
  <%/FOREACH%>

  <%unless($total_events){%>
  <CENTER><I>No Events</I>
  <% } %>

[DanO]

Try just adding a "$" to the string you used in your original conditional to make it a variable like it should be.

<%if ($total_events == 5) {

JFYI

Dan O.

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

[snooty1972]

We're getting there. I did as you suggested and added to $ to signify it is a variable. I also changed the total events declaration from 5 to 0 so that the count starts at 0 rather than 5. The result is nearly there. It now only shows 4 events BUT is also has the error message "Label not found for "last EVENTLOOP" at (eval 1) line 12.". This obviously means that the eventloop is working but something else is wrong that wasn't being round previously.

Here's the new nealy working code:

<%
&getEvents( {'duration'=>$in{duration}} );
$DAY = $Grid->{'grid'}->

• ->
• ;
  $EVENTS = $DAY->{events};
  $total_events=0;
  %>
  <%EVENTLOOP:%>
  <%FOREACH EVENTLIST%>
  <%IF EVENTS EXIST%>
  <BR>
  <%FOREACH EVENT%>
  <%$total_events++;%>
  <%if ($total_events == 5) { last EVENTLOOP; } %>
  <img src="http://www.regenthall.co.uk/regent/images/plus.gif" width=15 height=9 border=0>
  <A class=title HREF="<%=$CGI_URL_QUERYSTRING%>calendar=&template=&view=Event&event_id=<%=$EVENT->{'details'}->{'id'}%>"><%= $EVENT->{'details'}->{'title'} %></A><BR>
  <% if ($EVENT->{'schedule'}->{'start_time'}) { %><%= $EVENT->{'schedule'}->{'start_time'} %><% } %>

  <% if ($EVENT->{'schedule'}->{'start_time'}) { %> : <% } %>
  <%=$DAY->{dayname}%>, <%=$DAY->{monthname}%> <%=$DAY->{dd}%><BR>
  <%/FOREACH%>
  <%/IF%>
  <%/FOREACH%>

  <%unless($total_events){%>
  <CENTER><I>No Events</I>
  <% } %>

[delicia]

mine has the comparison after the event is printed and uses 'exit'. don't  a lot about perl; i got this from an old message here:

      <%FOREACH EVENT%>
         •
         <% $total_trueevents++;%>
         <% if ($EVENT->{'schedule'}->{'start_time'}) { %><%= $EVENT->{'schedule'}->{'start_time'} %><% } %>
         <% if ($EVENT->{'schedule'}->{'end_time'}) { %>-<%= $EVENT->{'schedule'}->{'end_time'} %><% } %>
         <% if ($EVENT->{'schedule'}->{'start_time'}) { %> : <% } %>
         <%= $EVENT->{'details'}->{'title'} %><BR>
         <% if($total_trueevents >= $max_events) {
            exit();
         }%>
      <%/FOREACH%>

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

[DanO]

** Label not found for "last EVENTLOOP"

Remove the line

<%EVENTLOOP:%>

(I don't know that such structure is supported - or correct)

Change "last EVENTLOOP;" to just

last;

** It now only shows 4 events **

Change the conditional ($total_events == 5) to

($total_events > 5)

OR move that line further down in the routine like before the first <%/FOREACH%>

Dan O.

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

[snooty1972]

Thanks Dan. That all works perfectly now. Really appreciated!

[DanO]

Glad to hear it.

Dan O.

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