Skip to:
Content
Pages
Categories
Search
Top
Bottom

Time Offset still present in BP 1.2.6

  • @paulhastings0

    Participant

    Running WP 3.0.1 with a child theme of the BP default, just upgraded to 1.2.6 and I’ve confirmed that the GMT time lag still exists for private messaging.

    I believe Ticket 2203 was supposed to fix this and was committed to the 1.2.6 release. Help?

Viewing 25 replies - 1 through 25 (of 33 total)
  • @hnla

    Participant

    I really hope you’re wrong about that as this ongoing issue was meant to have been corrected once and for all in 1.2.6. :(

    @paulhastings0

    Participant

    I’m afraid so.

    But if my memory doesn’t betray me, I think it’s a different part of the private messaging system that’s showing the wrong time now.

    @hnla

    Participant

    it’s displaying utc time by the looks of things at least my time stamp is an hour behind our present utc +1

    @hnla

    Participant

    checking and on the dev install I was working on I hadn’t set local time in WP settings

    @djpaul

    Keymaster

    Let me know what you discover, hnla

    @hnla

    Participant

    Not looking good having set my local time correctly and WP settings showing UTC and my local time messaging time stamps have now jumped ahead of my local time by 1 hour

    @djpaul

    Keymaster

    Alas, both of you are correct.

    @hnla

    Participant

    trying to track down bp_core_current_time() my batch find locates references to it but I can’t find actual parent function

    @djpaul

    Keymaster

    bp core, line 1312

    @hnla

    Participant

    cool cheers.

    this is/was the relevant ticket I guess?

    https://trac.buddypress.org/ticket/2203

    @djpaul

    Keymaster

    Yes

    @hnla

    Participant

    Thinking the issue lies with:

    /messages/messages-loop.php

    bp_message_thread_last_post_date()

    function lives in:
    /buddypress/bp-messages/bp-messages-templatetags.php: line 288

    would guess at the filter

    bp_format_time(get_date_from_gmt(

    is possibly to blame but will have to see if the ticket sheds light on it

    @hnla

    Participant

    Fix is simply that

    bp_format_time(get_date_from_gmt(

    should have been changed but likely missed to:

    bp_core_current_time(get_date_from_gmt(

    @hnla

    Participant

    Nope that isn’t correct :(

    can’t be ‘current’ time

    @djpaul

    Keymaster

    I think you’re half right – testing a fix.

    @hnla

    Participant

    I’m in the right area but run out of things to test tried bp_core_time_since but that we know works but returns unformatted plain time since and that doesn’t look correct so it must be the get_date_from_gmt function ?

    @djpaul

    Keymaster

    @hnla

    Participant

    That seems to have done the trick paul

    that strtotime is one of those php functions I never seem to remember

    @djpaul

    Keymaster

    If @paulhastings0 can confirm it works, I’ll put it in.

    @hnla

    Participant

    @paulhastings0 @paulhastings0 @paulhastings0

    Test please, you dragged this up :)

    @pisanojm

    Participant

    @djpaul @hnla

    I can confirm that this works… I had exact same issue with timestamp OMGPIZZA patch fixed it.

    @djpaul

    Keymaster

    Cool. It’s in the 1.2 branch now. It’s not up to me but I suspect we may do another release in due course to put this in and any other fixes we find.

    @pisanojm

    Participant

    @djpaul I agree… probably a 1.2.61 or something… It seems like there are a couple of other little “bugs” that should have been cleared cropping up as I read through the forums… Thanks for the fix.

    @paulhastings0

    Participant

    @hnla @djpaul So sorry yall, I’ve been at a conference that past few days with limited web access and then had an all-day meeting today. I’ll take a spin with it tomorrow. :)

    @paulhastings0

    Participant

    @djpaul @hnla So far so good. You mentioned that this is now in branch, but does that mean it’ll make it into 1.2.7 or do we need to do something more in the Trac?

Viewing 25 replies - 1 through 25 (of 33 total)
  • The topic ‘Time Offset still present in BP 1.2.6’ is closed to new replies.
Skip to toolbar