Support

May 17, 2019 at 2:43 am

Hi there,

I’ve removed the code from your opening post as its much easier to read on the gist, so no need for both.

The reason your getting different ticket prices is this code:

$tickets = EEH_Event_View::event_tickets_available( $post->ID );
// grab first ticket from array
$ticket = reset( $tickets );`

Pulls all of the tickets from the event and then you use the ‘first’ ticket from the array, but that won’t necessarily be the ticket you want. IIRC the ticket will be pulled based on the ticket ID ordered ASC.

EE doesn’t include a price in the table by default, so this is custom and not something we can support, but I can point you in the direction I would go. When you pull $tickets loop over them to find your non member ticket (how you do this is up to you) and then set $ticket to the current ticket in the loop and break out of it.

Side note, anywhere you have this:

if ( ! ($event->is_sold_out() || $event->is_sold_out( TRUE ) ) ) {

Will be using a fair amount of resources as the second condition is recalulating if the sold values have changed, you can generally swap that out for:

if ( ! ($event->is_sold_out() ) {

Which will reduce the load on the server when loading that page.

May 17, 2019 at 1:37 pm

Any idea why?

Yeah, my apologies, its my bad. This:

if ( ! ($event->is_sold_out() ) {

Needs to be:

if ( ! $event->is_sold_out() ) {

I missed removing the additional bracket.

For the pricing question, would it be possible for you to suggest the code to replace? The ticket should be easily identifiable – the one that doesn’t have any wp-user capability condition or the one without a discount.

That sounds like 2 conditions? The second isn’t as easy to detemine as the first.

Using a ticket that doesn’t have a capability set on it:

foreach($tickets as $temp_ticket) {
    if($temp_ticket instanceof EE_Ticket) {
        $cap_required = $temp_ticket->get_extra_meta('ee_ticket_cap_required', true);
        if(!$cap_required) {
            $ticket = $temp_ticket;
            break;
        }
    }
}

The loops over the tickets and the first it find that does not have a capability set on it will be the ticket the template uses, put that in place of:

// grab first ticket from array
$ticket = reset( $tickets );

May 17, 2019 at 2:55 pm

Not my day today!

Please try again with the code above (I’ve updated the post)