Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUN-1346: Remove database storage for "Node Filters" #8558

Merged
merged 53 commits into from
Nov 6, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
53 commits
Select commit Hold shift + click to select a range
edc8249
remove support for named node filters
gschueler Sep 14, 2023
d79c4b9
gsp: remove node filters
gschueler Sep 14, 2023
9874315
remove node filters from user Model type
gschueler Sep 14, 2023
3d2535a
remove node filter domain class
gschueler Sep 14, 2023
9c40397
remove node filter behavior from knockout/gsp front end code
gschueler Sep 15, 2023
b876518
cleanup: unused
gschueler Sep 15, 2023
122c154
cleanup: unused
gschueler Sep 15, 2023
c314626
remove node filter name
gschueler Sep 15, 2023
4039590
fix: correctly pass filter name param
gschueler Sep 15, 2023
639a0db
use local storage to manage node filters
gschueler Sep 15, 2023
64fcb44
cleanup: remove unused
gschueler Sep 15, 2023
24e4b05
add i18n text
gschueler Sep 15, 2023
ea7c0cb
cleanup db migration for removed table
gschueler Sep 15, 2023
57d2ad6
remove test for removed domain class
gschueler Sep 15, 2023
e036ca0
remove unneeded wrapper span
gschueler Sep 18, 2023
d3394b9
define mutation observer that can add ui sockets to new dom nodes
gschueler Sep 18, 2023
5e5bdfd
use ui-socket mutation observer for workflow editor
gschueler Sep 18, 2023
76e9433
remove filterName
gschueler Sep 18, 2023
85cde12
split selectNodeFilterLink to make it easier to call with data
gschueler Sep 18, 2023
34707d7
fix: filters should not always append
gschueler Sep 18, 2023
c855dd2
fix some typescript and incorrect function calls
gschueler Sep 18, 2023
9efb426
define node filter input ui sockets
gschueler Sep 18, 2023
3a9578d
some dom structure and css fixes
gschueler Sep 18, 2023
6e6cb96
Use Vue node filter input for Adhoc page
gschueler Sep 18, 2023
1bd99b1
use ui-socket for node filters on job edit/job reference step
gschueler Sep 19, 2023
632d17e
use ui-socket for node filter on job show page
gschueler Sep 19, 2023
a632e03
use ui-socket for node filter input on nodes page
gschueler Sep 19, 2023
b6f9ee2
cleanup: remove unnecessary attribute
gschueler Sep 19, 2023
3e6e98f
cleanup: ts
gschueler Sep 19, 2023
7d8098d
fix typescript errors
gschueler Sep 19, 2023
702758f
fix: selector, a changed to button
gschueler Sep 19, 2023
9e59d10
remove job filters from rdUser datatype
gschueler Sep 19, 2023
4d64e56
remove job filters data from frontend and controllers
gschueler Sep 19, 2023
95fc10c
remove ScheduledExecutionFilter domain class
gschueler Sep 19, 2023
ec987fe
remove db migration for scheduled_execution_filter
gschueler Sep 19, 2023
6b1ff72
remove saved job filters from GUI
gschueler Sep 19, 2023
ed1a625
job filters: skip constraint unless table exists
gschueler Sep 20, 2023
f104267
remove event filter storage endpoints from controller
gschueler Sep 20, 2023
1ef84ad
remove stored event filter urls
gschueler Sep 20, 2023
e2de1b5
add local storage for event filters
gschueler Sep 20, 2023
fc43427
don't pass filterName as query param
gschueler Sep 20, 2023
680608f
use local storage for activity filters
gschueler Sep 20, 2023
6b58638
update for new stored filter data type
gschueler Sep 20, 2023
f81498b
remove getReportFilters from RdUser interface
gschueler Sep 20, 2023
216c6e0
remove: unused
gschueler Sep 20, 2023
8a930d4
remove db loading of report filters
gschueler Sep 20, 2023
bf140a3
remove ReportFilter domain class
gschueler Sep 20, 2023
2c8b6ab
cleanup: wrong id
gschueler Sep 20, 2023
10c1cf3
remove debugger
gschueler Sep 25, 2023
0c955f8
update db migrations
gschueler Oct 2, 2023
2c08140
remove the tables that should not exist
gschueler Oct 2, 2023
89f439e
remove changeset for node_filter
gschueler Oct 31, 2023
3aa4f96
add validCheckSum
gschueler Oct 31, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,5 @@ public interface RdUser {
Date getLastLogout();
String getLastSessionId();
String getLastLoggedHostName();
Object getJobfilters();
Object getNodefilters();
Object getReportfilters();

}
9 changes: 1 addition & 8 deletions rundeckapp/grails-app/assets/javascripts/framework/adhoc.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ function runFormSubmit(elem) {
if (running || !$F('runFormExec')) {
return false;
}
if (!nodeFilter.filter() && !nodeFilter.filterName()) {
if (!nodeFilter.filter() ) {
//no node filter
return false;
}
Expand Down Expand Up @@ -290,13 +290,6 @@ function init() {

adhocCommand = new AdhocCommand({commandString:pageParams.runCommand}, nodeFilter);

//show selected named filter
nodeFilter.filterName.subscribe(function (val) {
if (val) {
jQuery('a[data-node-filter-name]').removeClass('active');
jQuery('a[data-node-filter-name=\'' + val + '\']').addClass('active');
}
});
nodeFilter.total.subscribe(function(val){
if (val && val != "0" && !running) {
adhocCommand.canRun(true);
Expand Down
31 changes: 8 additions & 23 deletions rundeckapp/grails-app/assets/javascripts/framework/nodes.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ var pageParams={};
* @param elem
*/
function expandResultNodes(page,elem){
loadNodeFilter(null,nodeFilter.filter(),nodeFilter.filterAll(),elem,page);
loadNodeFilter(nodeFilter.filter(),nodeFilter.filterAll(),elem,page);
}
/**
* load either filter string or saved filter
Expand All @@ -51,37 +51,29 @@ function expandResultNodes(page,elem){
* @param elem target element
* @param page number to load
*/
function loadNodeFilter(filterName, filterString,filterAll,elem,page) {
function loadNodeFilter( filterString,filterAll,elem,page) {
jQuery('.nodefilterlink').removeClass('active');
if (!page) {
page = 0;
}
if (!elem) {
elem = 'nodelist';
}
if(!filterName&&!filterString&&null==filterAll){
filterName=nodeFilter.filterName();
if(!filterString&&null==filterAll){
filterString=nodeFilter.filter();
filterAll=nodeFilter.filterAll();
}
if(!filterName && !filterString){
if(!filterString){
//if blank input and no filtername selected, do nothing
return;
}
nodespage = page;
var view = page == 0 ? 'table' : 'tableContent';
var data = filterName? {filterName: filterName} : {filter: filterString};
var data = {filter: filterString};
data.nodeExcludePrecedence='true';
if(filterName){
jQuery('a[data-node-filter-name=\''+filterName+'\']').addClass('active');
jQuery('.hiddenNodeFilter').val(filterString);
jQuery('.hiddenNodeFilterName').val(filterName);
}else{
jQuery('.hiddenNodeFilter').val(filterString );
jQuery('.hiddenNodeFilterName').val('');
}
jQuery('.hiddenNodeFilter').val(filterString );
jQuery('.hiddenNodeFilterName').val('');
nodeFilter.filterAll(filterAll);
nodeFilter.filterName(filterName);
nodeFilter.filter(filterString);
nodeFilter.loading(true);
_updateMatchedNodes(data,elem,pageParams.project,false,{view:view,expanddetail:true,inlinepaging:true,
Expand Down Expand Up @@ -169,7 +161,7 @@ var nodeSummary;
*/
function pageLoad(){
var pagestate;
if(filterParams.filterName || filterParams.filter|| filterParams.filterAll ){
if( filterParams.filter|| filterParams.filterAll ){
nodeFilter.setPageParams(filterParams);
nodeFilter.updateMatchedNodes();
pagestate=nodeFilter.getPageParams();
Expand Down Expand Up @@ -208,13 +200,6 @@ function init() {
nodeSummary:nodeSummary
}));
ko.applyBindings(nodeFilter);
//show selected named filter
nodeFilter.filterName.subscribe(function (val) {
if (val) {
jQuery('a[data-node-filter-name]').removeClass('active');
jQuery('a[data-node-filter-name=\'' + val + '\']').addClass('active');
}
});
nodeFilter.loading.subscribe(function(val){
//select filter results tab whenever loading output
jQuery('#tab_link_result > a').tab('show');
Expand Down
149 changes: 0 additions & 149 deletions rundeckapp/grails-app/assets/javascripts/jobFiltersKO.js

This file was deleted.

1 change: 0 additions & 1 deletion rundeckapp/grails-app/assets/javascripts/menu/jobs.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
//= require knockout-foreachprop
//= require nodeFiltersKO
//= require executionOptions
//= require jobFiltersKO
//= require knockout-onenter
//= require knockout-node-filter-link
//= require menu/job-remote-options
Expand Down