Skip to content

Commit

Permalink
Remove unused preserve state attribute from grids
Browse files Browse the repository at this point in the history
  • Loading branch information
guerler committed Dec 10, 2017
1 parent 7624ce2 commit 0f7bc19
Show file tree
Hide file tree
Showing 11 changed files with 2 additions and 53 deletions.
18 changes: 1 addition & 17 deletions lib/galaxy/web/framework/helpers/grids.py
Expand Up @@ -32,7 +32,6 @@ class Grid(object):
# Any columns that are filterable (either standard or advanced) should have a default value set in the default filter.
default_filter = {}
default_sort_key = None
preserve_state = False
use_paging = False
num_rows_per_page = 25
num_page_links = 10
Expand Down Expand Up @@ -69,14 +68,6 @@ def __call__(self, trans, **kwargs):
# default_filter is a dictionary that provides a default set of filters based on the grid's columns.
base_filter = self.default_filter.copy()
base_sort_key = self.default_sort_key
if self.preserve_state:
pref_name = text_type(self.__class__.__name__ + self.cur_filter_pref_name)
if pref_name in trans.get_user().preferences:
saved_filter = loads(trans.get_user().preferences[pref_name])
base_filter.update(saved_filter)
pref_name = text_type(self.__class__.__name__ + self.cur_sort_key_pref_name)
if pref_name in trans.get_user().preferences:
base_sort_key = loads(trans.get_user().preferences[pref_name])
# Build initial query
query = self.build_initial_query(trans, **kwargs)
query = self.apply_query_filter(trans, query, **kwargs)
Expand Down Expand Up @@ -224,14 +215,7 @@ def loads_recurse(item):
# There are some places in grid templates where it's useful for a grid
# to have its current filter.
self.cur_filter_dict = cur_filter_dict
# Preserve grid state: save current filter and sort key.
if self.preserve_state:
pref_name = text_type(self.__class__.__name__ + self.cur_filter_pref_name)
trans.get_user().preferences[pref_name] = text_type(dumps(cur_filter_dict))
if sort_key:
pref_name = text_type(self.__class__.__name__ + self.cur_sort_key_pref_name)
trans.get_user().preferences[pref_name] = text_type(dumps(sort_key))
trans.sa_session.flush()

# Log grid view.
context = text_type(self.__class__.__name__)
params = cur_filter_dict.copy()
Expand Down
5 changes: 0 additions & 5 deletions lib/galaxy/webapps/galaxy/controllers/admin.py
Expand Up @@ -154,7 +154,6 @@ def get_value(self, trans, grid, user):
grids.GridColumnFilter("All", args=dict(deleted='All'))
]
num_rows_per_page = 50
preserve_state = False
use_paging = True

def get_current_item(self, trans, **kwargs):
Expand Down Expand Up @@ -255,7 +254,6 @@ def get_value(self, trans, grid, role):
grids.GridColumnFilter("All", args=dict(deleted='All'))
]
num_rows_per_page = 50
preserve_state = False
use_paging = True

def apply_query_filter(self, trans, query, **kwargs):
Expand Down Expand Up @@ -335,7 +333,6 @@ def get_value(self, trans, grid, group):
grids.GridColumnFilter("All", args=dict(deleted='All'))
]
num_rows_per_page = 50
preserve_state = False
use_paging = True


Expand Down Expand Up @@ -447,7 +444,6 @@ def get_value(self, trans, grid, quota):
grids.GridColumnFilter("All", args=dict(deleted='All'))
]
num_rows_per_page = 50
preserve_state = False
use_paging = True


Expand Down Expand Up @@ -499,7 +495,6 @@ def get_value(self, trans, grid, tool_version):
standard_filters = []
default_filter = {}
num_rows_per_page = 50
preserve_state = False
use_paging = True

def build_initial_query(self, trans, **kwd):
Expand Down
1 change: 0 additions & 1 deletion lib/galaxy/webapps/galaxy/controllers/dataset.py
Expand Up @@ -84,7 +84,6 @@ def get_accepted_filters(self):
]
standard_filters = []
default_filter = dict(name="All", deleted="False", tags="All")
preserve_state = False
use_paging = True
num_rows_per_page = 50

Expand Down
1 change: 0 additions & 1 deletion lib/galaxy/webapps/galaxy/controllers/forms.py
Expand Up @@ -39,7 +39,6 @@ def get_value(self, trans, grid, user):
model_class = model.FormDefinitionCurrent
default_sort_key = "-update_time"
num_rows_per_page = 50
preserve_state = True
use_paging = True
default_filter = dict(deleted="False")
columns = [
Expand Down
1 change: 0 additions & 1 deletion lib/galaxy/webapps/galaxy/controllers/history.py
Expand Up @@ -130,7 +130,6 @@ def sort(self, trans, query, ascending, column_name=None):
]
default_filter = dict(name="All", deleted="False", tags="All", sharing="All")
num_rows_per_page = 15
preserve_state = False
use_paging = True
info_text = "Histories that have been deleted for more than a time period specified by the Galaxy administrator(s) may be permanently deleted."

Expand Down
1 change: 0 additions & 1 deletion lib/galaxy/webapps/reports/controllers/jobs.py
Expand Up @@ -222,7 +222,6 @@ def filter(self, trans, user, query, column_filter):
standard_filters = []
default_filter = {'specified_date': 'All'}
num_rows_per_page = 50
preserve_state = False
use_paging = True

def build_initial_query(self, trans, **kwd):
Expand Down
1 change: 0 additions & 1 deletion lib/galaxy/webapps/reports/controllers/workflows.py
Expand Up @@ -112,7 +112,6 @@ def filter(self, trans, user, query, column_filter):
standard_filters = []
default_filter = {'specified_date': 'All'}
num_rows_per_page = 50
preserve_state = False
use_paging = True

def build_initial_query(self, trans, **kwd):
Expand Down
18 changes: 1 addition & 17 deletions lib/galaxy/webapps/reports/framework/grids.py
Expand Up @@ -35,7 +35,6 @@ class Grid(object):
# Any columns that are filterable (either standard or advanced) should have a default value set in the default filter.
default_filter = {}
default_sort_key = None
preserve_state = False
use_paging = False
num_rows_per_page = 25
num_page_links = 10
Expand Down Expand Up @@ -72,14 +71,6 @@ def __call__(self, trans, **kwargs):
# default_filter is a dictionary that provides a default set of filters based on the grid's columns.
base_filter = self.default_filter.copy()
base_sort_key = self.default_sort_key
if self.preserve_state:
pref_name = text_type(self.__class__.__name__ + self.cur_filter_pref_name)
if pref_name in trans.get_user().preferences:
saved_filter = loads(trans.get_user().preferences[pref_name])
base_filter.update(saved_filter)
pref_name = text_type(self.__class__.__name__ + self.cur_sort_key_pref_name)
if pref_name in trans.get_user().preferences:
base_sort_key = loads(trans.get_user().preferences[pref_name])
# Build initial query
query = self.build_initial_query(trans, **kwargs)
query = self.apply_query_filter(trans, query, **kwargs)
Expand Down Expand Up @@ -227,14 +218,7 @@ def loads_recurse(item):
# There are some places in grid templates where it's useful for a grid
# to have its current filter.
self.cur_filter_dict = cur_filter_dict
# Preserve grid state: save current filter and sort key.
if self.preserve_state:
pref_name = text_type(self.__class__.__name__ + self.cur_filter_pref_name)
trans.get_user().preferences[pref_name] = text_type(dumps(cur_filter_dict))
if sort_key:
pref_name = text_type(self.__class__.__name__ + self.cur_sort_key_pref_name)
trans.get_user().preferences[pref_name] = text_type(dumps(sort_key))
trans.sa_session.flush()

# Log grid view.
context = text_type(self.__class__.__name__)
params = cur_filter_dict.copy()
Expand Down
1 change: 0 additions & 1 deletion lib/tool_shed/galaxy_install/grids/admin_toolshed_grids.py
Expand Up @@ -215,7 +215,6 @@ def get_accepted_filters(self):
standard_filters = []
default_filter = dict(deleted="False")
num_rows_per_page = 50
preserve_state = False
use_paging = False

def build_initial_query(self, trans, **kwd):
Expand Down
6 changes: 0 additions & 6 deletions lib/tool_shed/grids/repository_grids.py
Expand Up @@ -62,7 +62,6 @@ def get_value(self, trans, grid, category):
operations = []
standard_filters = []
num_rows_per_page = 50
preserve_state = False
use_paging = False


Expand Down Expand Up @@ -236,7 +235,6 @@ def get_value(self, trans, grid, repository):
standard_filters = []
default_filter = dict(deleted="False")
num_rows_per_page = 50
preserve_state = False
use_paging = False

def build_initial_query(self, trans, **kwd):
Expand Down Expand Up @@ -358,7 +356,6 @@ def get_value(self, trans, grid, repository_metadata):
standard_filters = []
default_filter = {}
num_rows_per_page = 50
preserve_state = False
use_paging = False

def build_initial_query(self, trans, **kwd):
Expand Down Expand Up @@ -457,7 +454,6 @@ class RepositoriesByUserGrid(RepositoryGrid):
standard_filters = []
default_filter = dict(deleted="False")
num_rows_per_page = 50
preserve_state = False
use_paging = False

def build_initial_query(self, trans, **kwd):
Expand Down Expand Up @@ -978,7 +974,6 @@ def get_value(self, trans, grid, repository_metadata):
standard_filters = []
default_filter = dict(malicious="False")
num_rows_per_page = 50
preserve_state = False
use_paging = False

def build_initial_query(self, trans, **kwd):
Expand Down Expand Up @@ -1298,7 +1293,6 @@ def get_value(self, trans, grid, category):
operations = []
standard_filters = []
num_rows_per_page = 50
preserve_state = False
use_paging = False


Expand Down
2 changes: 0 additions & 2 deletions lib/tool_shed/grids/repository_review_grids.py
Expand Up @@ -45,7 +45,6 @@ def get_value(self, trans, grid, component):
operations = []
standard_filters = []
num_rows_per_page = 50
preserve_state = False
use_paging = False


Expand Down Expand Up @@ -349,7 +348,6 @@ def get_value(self, trans, grid, review):
]
standard_filters = []
num_rows_per_page = 50
preserve_state = False
use_paging = False

def build_initial_query(self, trans, **kwd):
Expand Down

0 comments on commit 0f7bc19

Please sign in to comment.