Opened 11 years ago
Closed 11 years ago
#1647 closed defect (bug) (fixed)
Improve deprecation notice banner text generation.
| Reported by: | coffee2code | Owned by: | coffee2code |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Developer Hub | Keywords: | |
| Cc: |
Description
Currently, the text that appears in the deprecation notice banner for deprecated resources consists of two parts:
- The boilerplate
This {$content_type} has been deprecated. - And, if possible, an additional message referring visitors to an alternative resource e.g.
Use other_function() instead..
The message for part two is currently solely derived from the presence of a @see tag that ostensibly points to the alternative function, which is fine for many cases, but not in the case for others.
Proposed improvements:
- For some reason, the parser sometimes drops the ending '()' from functions and methods mentioned via
@see, so restore those for display if appropriate. - If the function does not specify a
@seetag, then attempt to use the message provided with the@deprecatedtag. (Note: the@seeis preferred over the more direct@deprecatedbecause the generated message is translatable whereas the@deprecatedmessage is not.) - If the
@deprecatedmessage is used, append 'instead.' to the string if not already present. Many deprecated notices are of the syntaxUse other_function()rather than the preferredUse other_function() instead.. This makes the message consistent regardless of how it was derived.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
In 2816: