Project

General

Profile

Actions

Patch #15118

closed

Deprecate and rename rss_* methods to atom_* methods

Added by Mischa The Evil over 10 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

This issue is extracted from #13460-2 after 2.4 feature freeze.

Relevant history:

Daniel Felix wrote in #13460-2 (note-2):

  • File Attachment:functional_renaming_rss_to_atom.diff added

I'll provide two patches.
[...]
Number 2: functional renaming. Deprecation of old rss_* methods and redefining them as atom_* methods. Old methods are still available and give deprecation warnings. Please test Number 2, too. But it seems to work in the first test.

Jean-Philippe Lang wrote in #13460-7 (note-7):

I've committed the translations in r11654 but I think it's a bit late to change the methods for 2.3.0.


Files

functional_renaming_rss_to_atom.diff (26.9 KB) functional_renaming_rss_to_atom.diff Patch authored by Daniel Felix (transported from #13460-2) Mischa The Evil, 2013-10-14 01:31

Related issues

Related to Redmine - Patch #13460: All translations: RSS -> AtomClosed

Actions
Related to Redmine - Patch #36806: Remove rss_* deprecated methodsClosedMarius BĂLTEANU

Actions
Actions #1

Updated by Mischa The Evil over 10 years ago

The affected methods are MyController#reset_rss_key, User#rss_key and User.find_by_rss_key.

Daniel Felix wrote in #13460-8 (note-8):

Hopefully the way I use to deprecate those methods is good enough for the implementation. :-)

Pretty much I think, though some of the deprecation warnings used in the patch are not correct:
  1. the warning for MyController#reset_rss_key reads:
    "My#reset_rss_key is deprecated and will be removed in Redmine 3.0. Please use #reset_atom_key instead."
    this should be
    "MyController#reset_rss_key is deprecated and will be removed in Redmine 3.0. Please use MyController#reset_atom_key instead."
  2. the warning for User#rss_key reads:
    "User.rss_key is deprecated and will be removed in Redmine 3.0. Please use User.atom_key instead."
    this should be
    "User#rss_key is deprecated and will be removed in Redmine 3.0. Please use User#atom_key instead."
For 1:
  • My#reset_rss_key says that reset_rss_key is an instance method of My (suggesting that My is an existing model in the Redmine core, which isn't), whereas the change you've made is in the reset_rss_key instance method of MyController
  • I think it is better to include the class name when redirecting (in text obviously :) the user to the new, replacing method
For 2:
  • User.rss_key refers to a class method in the User class, while in the Redmine core rss_key is an instance method in the User class
Actions #2

Updated by Mischa The Evil over 10 years ago

  • Related to Patch #13460: All translations: RSS -> Atom added
Actions #3

Updated by Mischa The Evil over 2 years ago

  • Target version changed from Candidate for next major release to 5.0.0

Shall we finally properly deprecate this in 5.0.0 and remove it in subsequent minor 5.x.x release (e.g. 5.1.0 or 5.2.0)?

Actions #4

Updated by Go MAEDA over 2 years ago

Mischa The Evil wrote:

Shall we finally properly deprecate this in 5.0.0 and remove it in subsequent minor 5.x.x release (e.g. 5.1.0 or 5.2.0)?

I agree.

Actions #5

Updated by Mischa The Evil over 2 years ago

Go, do you need a new, rebased patch file or do you prefer to do the search and replace and the addition of the deprecation messages (modified according to my comments in note#1) locally yourself?

Actions #6

Updated by Marius BĂLTEANU about 2 years ago

  • File 0001-Deprecate-and-rename-rss_-methods-to-atom_-methods-1.patch added

Updated the patch for current trunk.

Actions #7

Updated by Marius BĂLTEANU about 2 years ago

  • File deleted (0001-Deprecate-and-rename-rss_-methods-to-atom_-methods-1.patch)
Actions #8

Updated by Marius BĂLTEANU about 2 years ago

  • Status changed from New to Resolved
  • Assignee set to Marius BĂLTEANU

Patch committed, thanks!

Actions #9

Updated by Marius BĂLTEANU about 2 years ago

  • Related to Patch #36806: Remove rss_* deprecated methods added
Actions #10

Updated by Marius BĂLTEANU about 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF