[Rt-commit] rt branch, 4.2/depricate-bookmarks, created. rt-4.1.8-303-ge9ea7e4

Todd Wade todd at bestpractical.com
Tue May 14 23:32:38 EDT 2013


The branch, 4.2/depricate-bookmarks has been created
        at  e9ea7e49903353ab78e07659b74082d08cd70ccb (commit)

- Log -----------------------------------------------------------------
commit e9ea7e49903353ab78e07659b74082d08cd70ccb
Author: Todd Wade <todd at bestpractical.com>
Date:   Tue May 14 12:13:39 2013 -0400

    mark __Bookmarks__ as deprecated
    
    __Bookmarks__ was to be removed in 4.2 but the user was never warned
    in 4.0, so marking the feature deprecated for 4.2 and will be fully
    removed later
    
    see issues#20297

diff --git a/share/html/Elements/ShowSearch b/share/html/Elements/ShowSearch
index fb9b043..524280d 100644
--- a/share/html/Elements/ShowSearch
+++ b/share/html/Elements/ShowSearch
@@ -131,7 +131,9 @@ foreach ( $SearchArg, $ProcessedSearchArg ) {
     elsif ( $_->{'Query'} =~ /__Bookmarks__/ ) {
         $_->{'Rows'} = 999;
 
-        # DEPRECATED: will be here for a while up to 3.10/4.0
+        # DEPRECATED: will be here for a while up to 4.4
+        RT->Deprecated( Remove => "4.4", Instead => "id = '__Bookmarked__'" );
+
         my @bookmarks = $session{'CurrentUser'}->UserObj->Bookmarks;
         my $query = join(" OR ", map " id = '$_' ", @bookmarks ) || 'id=0';
         $_->{'Query'} =~ s/__Bookmarks__/( $query )/g;
diff --git a/share/html/m/tickets/search b/share/html/m/tickets/search
index b5ae8e4..6d5704e 100644
--- a/share/html/m/tickets/search
+++ b/share/html/m/tickets/search
@@ -90,7 +90,9 @@ my $search;
         if ( $_->{'Query'} =~ /__Bookmarks__/ ) {
             $_->{'Rows'} = 999;
 
-            # DEPRECATED: will be here for a while up to 3.10/4.0
+            # DEPRECATED: will be here for a while up to 4.4
+            RT->Deprecated( Remove => "4.4", Instead => "id = '__Bookmarked__'" );
+
             my @bookmarks = $session{'CurrentUser'}->UserObj->Bookmarks;
             my $query = join(" OR ", map " id = '$_' ", @bookmarks ) || 'id=0';
             $_->{'Query'} =~ s/__Bookmarks__/( $query )/g;

-----------------------------------------------------------------------


More information about the Rt-commit mailing list