Opened 6 years ago

Closed 5 years ago

Last modified 4 years ago

#11933 closed (fixed)

Missing cached_db value in SESSION_ENGINE setting docs

Reported by: gabrielhurley Owned by: gabrielhurley
Component: Documentation Version: 1.1
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The docs don't list cached_db as a "valid value" in either ref/settings or topics/http/sessions.

The submitted patch corrects both instances and updates the .. versionadded:: 1.0 to .. versionchanged:: 1.1 for each.

Attachments (2)

docs_session_engine_cached_db.diff (1.2 KB) - added by gabrielhurley 6 years ago.
11933_correction.diff (1.4 KB) - added by gabrielhurley 5 years ago.
corrects cache_db to cached_db, patched against current trunk

Download all attachments as: .zip

Change History (12)

Changed 6 years ago by gabrielhurley

comment:1 Changed 6 years ago by gabrielhurley

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to gabrielhurley
  • Patch needs improvement unset
  • Status changed from new to assigned

comment:2 Changed 5 years ago by russellm

  • milestone set to 1.2
  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 5 years ago by russellm

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

(In [12569]) Fixed #11933 -- Added versionchanged marker for the cache_db session backend. Thanks to gabrielhurley for the report and patch.

comment:4 Changed 5 years ago by russellm

(In [12571]) [1.1.X] Fixed #11933 -- Added versionchanged marker for the cache_db session backend. Thanks to gabrielhurley for the report and patch.

Backport of r12569 from trunk.

comment:5 Changed 5 years ago by mb0@…

  • Patch needs improvement set
  • Resolution fixed deleted
  • Status changed from closed to reopened

sorry guys, the patch is incorrect the backend is called cached_db and not cache_db.

Changed 5 years ago by gabrielhurley

corrects cache_db to cached_db, patched against current trunk

comment:6 Changed 5 years ago by gabrielhurley

  • Patch needs improvement unset

fixed (again). corrections done against latest trunk [12949] so it applies cleanly. sorry 'bout that.

comment:7 Changed 5 years ago by russellm

  • Triage Stage changed from Accepted to Ready for checkin

comment:8 Changed 5 years ago by russellm

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

(In [13169]) Fixed #11933 -- Corrected the naming of the cached_db session backend. Thanks to mb0@… for the report and Gabriel Hurley for the patch.

comment:9 Changed 5 years ago by russellm

(In [13175]) [1.1.X] Fixed #11933 -- Corrected the naming of the cached_db session backend. Thanks to mb0@… for the report and Gabriel Hurley for the patch.

Backport of r13169 from trunk.

comment:10 Changed 4 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

Note: See TracTickets for help on using tickets.
Back to Top