Forums
-
Announcements
-
- 1059
- posts
-
-
General
-
- 6027
- posts
-
- 1187
- posts
-
- 40
- posts
-
- 403
- posts
-
-
JReviews 3
-
- 130
- posts
-
- 15
- posts
-
- 268
- posts
-
- 41
- posts
-
- 2527
- posts
-
- 201
- posts
-
- 10
- posts
-
- 79
- posts
-
- 77
- posts
-
-
JReviews Add-ons
-
- 110
- posts
-
- 115
- posts
-
- 351
- posts
-
- 115
- posts
-
- 568
- posts
-
- 4926
- posts
-
- 3082
- posts
-
- 116
- posts
-
- 1365
- posts
-
-
JReviews en Español
-
- 1163
- posts
-
-
JReviews 2.x
-
- 17492
- posts
-
- 891
- posts
-
- 40908
- posts
-
- 6524
- posts
-
- 2750
- posts
-
- 5757
- posts
-
- 724
- posts
-
- 373
- posts
-
- 411
- posts
-
-
Extensions & Templates
-
- 1677
- posts
-
-
Feature requests
-
- 4366
- posts
-
-
Freelance board
-
- 1971
- posts
-
- 133
- posts
-
-
3rd Party Solutions for JReviews
-
- 91
- posts
-
-
Recent possts
-
Perfect to use it within a module. -
By Matthew Bayliss · Posted
So this is what I've come up with based on the workaround idea I mentioned in the last post. It definitely works, just not sure if there's a better way? <?php $expired = $listing['Listing']['publish_down'] != NULL_DATE && strtotime($listing['Listing']['publish_down']) < time(); ?> <?php if($expired == "0"):?> <!–– listing template --!> <?php endif;?> -
By Matthew Bayliss · Posted
Yeah that sounds like what's happening. Only issue is that we need to be able to stop the listing outputting part way through the day as it relates to offers which we're not supposed to show after they've expired. Setting to end a day early would't work either as for big events they're only live for part of one day so they wouldn't show at all. Plus interest peaks closer to when the offer ends so again wouldn't want it to end early. We'd previously been using a butchered version of the joomla banner system to contain these but wanted to use the related listing functions of jreviews. What if we checked the current time in the module theme file and compared to the joomla expiration time field. We'd have the module on a short cache so there would be a bit of overlap (but we'd have that anyway with the cache, whichever way we did it) but means the database query could still be cached, right? -
Not necessarily. For performance reasons, JReviews doesn't include the current time in database queries. That's because if it did, then every time the page loaded the query would be different and MySQL is not able to cache it. So checking for expired listings is done based on the beginning of the day. So the queries look for listings that have an expiration date/time higher than the beginning of today. I guess this could turn into an issue if Joomla considers an article expiring at noon as expired, but JReviews allows it to show for the entire day since it was after all higher than the beginning of today. The only solution here, so as not to mess up with performance, would be to expire the listing one day before so that if the expiration is noon today, it will only show till the end of the previous day. Does any of this make sense and sounds like it could be the issue based on what you are seeing? There's also the issue with server time vs. real time, but hopefully, that's not something that's affecting this as well as there are built-in methods to take care of that.
-
By Matthew Bayliss · Posted
We've not upgraded to 3.x but it is the latest version of 2.7 (v2.7.26.1). So after doing a bit more digging it seems that they are expiring but the time is ignored - ie: ones expired yesterday are removed but ones expired earlier today are still showing. Is that by design?
-