Bug with Venus date display

Home Forums Calendar Products Events Calendar PRO Bug with Venus date display

Viewing 3 posts - 1 through 3 (of 3 total)
  • Author
    Posts
  • #13813
    Mike
    Participant

    Hi guys

    I’ve found a bug in the PRO version with the single-venue.php.

    Existing code:

    if (tribe_is_multiday()):

    Should be (grabbed from list.php):

    if (tribe_is_multiday() || !tribe_get_all_day()):

    The code out of the box does not display the state/end time for events which aren’t “all day events”.

    On a related note, there’s a LOT of duplication in list.php and single-venue.php. It would be great if the code could be consolidated and re-used. I’ve had to change stuff I want in both places so it looks/behaves the same.

    #13860
    Rob
    Member

    This is a good catch, Mike — thanks for pointing it out. Let me double check with the devs that there is no reason we intentionally did it that way (assuming there wasn’t though) and we’ll get this logged for 2.1 to resolve if needed.

    #13901
    Joey Kudish
    Member

    Hi Mike,

    Thanks for identifying this issue/bug. I’ve tested a few scenarios and I agree with you that the code should in fact read as you specified. I’ve logged this as an issue to fix for 2.1 (our upcoming release) and we’ll also take a look at simplifying/combining the code at that time.

    Thanks again for your suggestion and good developer’s eye and all the best!
    Joey

Viewing 3 posts - 1 through 3 (of 3 total)
  • The topic ‘Bug with Venus date display’ is closed to new replies.