Skip to:
Content

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#2735 closed defect (bug) (fixed)

Properly append group id to cache in BP_Groups_Template::the_group()

Reported by: cnorris23 Owned by:
Milestone: 1.5 Priority: normal
Severity: Version: 1.2.6
Component: Groups Keywords: has-patch
Cc:

Description

Noticed this when I received numerous cache slam errors after activating W3 Total Cache with APC object caching. Checked trunk to see if it needed patching too, but this data isn't cached in trunk. I don't know if this is intentional, so I've left it unpatched for now.

Attachments (1)

2735.001.diff (595 bytes) - added by cnorris23 8 years ago.

Download all attachments as: .zip

Change History (5)

@cnorris23
8 years ago

#1 @DJPaul
8 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

Good spot. Looking through SVN, groups_group_nouserdata_ hasn't been retrieved since 1.1.3; it's been set and cleared on every version since, so I took it out of trunk recently.

#2 @cnorris23
8 years ago

  • Resolution wontfix deleted
  • Status changed from closed to reopened

Should we remove the code for the 1.2.x branch then, or at least fix the group id? Obviously it can be patched by hand after an update, but that would be one less thing to remember if it were already fixed.

#3 @johnjamesjacoby
8 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [3422]) Fixes #2735 (branch) props cnorris23

#4 @johnjamesjacoby
8 years ago

Fixed in branch for now. Can look at this more later if it causes any more headaches.

Note: See TracTickets for help on using tickets.