Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#18131 closed Bug (fixed)

ContentTypeManager.get_for_id not documented

Reported by: sir_sigurd Owned by: nobody
Component: Documentation Version: 1.4
Severity: Normal Keywords:
Cc: sir_sigurd, timograham@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by kmtracey)

get_for_id is not listed among ContentTypeManager methods.

The only mention of caching is in description of clear_cache method, but documentation should say that get_* methods are cached and using them is preferred way to get ContentType.

Attachments (1)

18131.diff (746 bytes) - added by timo 3 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 3 years ago by sir_sigurd

  • Component changed from Uncategorized to Documentation
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
Version 0, edited 3 years ago by sir_sigurd (next)

comment:2 Changed 3 years ago by jezdez

  • Triage Stage changed from Unreviewed to Accepted
  • Type changed from Uncategorized to Bug

comment:3 Changed 3 years ago by timo

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

comment:4 Changed 3 years ago by sir_sigurd

  • Cc sir_sigurd added
  • Resolution fixed deleted
  • Status changed from closed to reopened

I see no changes there for fixing this bug.

comment:5 Changed 3 years ago by kmtracey

  • Description modified (diff)

Corrected name of undocumented method in original description.

Changed 3 years ago by timo

comment:6 Changed 3 years ago by timo

  • Cc timograham@… added
  • Has patch set
  • Summary changed from insufficient info about ContentTypeManager to ContentTypeManager.get_for_id not documented

comment:7 Changed 3 years ago by Tim Graham <timograham@…>

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

In 93e6733e4cbbdad34f1f0f59303ae01f577e4e58:

Fixed #18131 - Documented ContentTypeManager.get_for_id; thanks sir_sigurd for the report.

comment:8 Changed 3 years ago by Tim Graham <timograham@…>

In 421ce44e8bb7386d1f5826d0473a3e24f6075d35:

[1.4.x] Fixed #18131 - Documented ContentTypeManager.get_for_id; thanks sir_sigurd for the report.

Backport of 93e6733e4cbbdad34f1f0f59303ae01f577e4e58 from master.

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