From c9ce5962a61d648f95d03a270ae485449f139ab7 Mon Sep 17 00:00:00 2001 From: BlueTeck Date: Sat, 10 Oct 2015 17:23:41 +0200 Subject: #1340 add append/replace option to dustom filters --- app/Controller/Customfilter.php | 4 ++++ app/Model/CustomFilter.php | 3 ++- app/Schema/Mysql.php | 7 ++++++- app/Schema/Postgres.php | 7 ++++++- app/Schema/Sqlite.php | 7 ++++++- app/Template/custom_filter/add.php | 4 +++- app/Template/custom_filter/edit.php | 6 ++++-- app/Template/custom_filter/index.php | 10 +++++++++- app/Template/project/filters.php | 2 +- 9 files changed, 41 insertions(+), 9 deletions(-) (limited to 'app') diff --git a/app/Controller/Customfilter.php b/app/Controller/Customfilter.php index c403cb4b..80d18a17 100644 --- a/app/Controller/Customfilter.php +++ b/app/Controller/Customfilter.php @@ -115,6 +115,10 @@ class Customfilter extends Base if (! isset($values['is_shared'])) { $values += array('is_shared' => 0); } + + if (! isset($values['append'])) { + $values += array('append' => 0); + } list($valid, $errors) = $this->customFilter->validateModification($values); diff --git a/app/Model/CustomFilter.php b/app/Model/CustomFilter.php index 2c485247..b7ccdf99 100644 --- a/app/Model/CustomFilter.php +++ b/app/Model/CustomFilter.php @@ -40,7 +40,8 @@ class CustomFilter extends Base self::TABLE.'.project_id', self::TABLE.'.filter', self::TABLE.'.name', - self::TABLE.'.is_shared' + self::TABLE.'.is_shared', + self::TABLE.'.append' ) ->asc(self::TABLE.'.name') ->join(User::TABLE, 'id', 'user_id') diff --git a/app/Schema/Mysql.php b/app/Schema/Mysql.php index 37894dbd..6ece2785 100644 --- a/app/Schema/Mysql.php +++ b/app/Schema/Mysql.php @@ -6,7 +6,12 @@ use PDO; use Core\Security; use Model\Link; -const VERSION = 90; +const VERSION = 91; + +function version_91($pdo) +{ + $pdo->exec("ALTER TABLE custom_filters ADD COLUMN append TINYINT(1) DEFAULT 0"); +} function version_90($pdo) { diff --git a/app/Schema/Postgres.php b/app/Schema/Postgres.php index 2a446e40..d4fae317 100644 --- a/app/Schema/Postgres.php +++ b/app/Schema/Postgres.php @@ -6,7 +6,12 @@ use PDO; use Core\Security; use Model\Link; -const VERSION = 70; +const VERSION = 71; + +function version_71($pdo) +{ + $pdo->exec("ALTER TABLE custom_filters ADD COLUMN append BOOLEAN DEFAULT '0'"); +} function version_70($pdo) { diff --git a/app/Schema/Sqlite.php b/app/Schema/Sqlite.php index 633c90a7..2b9ceadc 100644 --- a/app/Schema/Sqlite.php +++ b/app/Schema/Sqlite.php @@ -6,7 +6,12 @@ use Core\Security; use PDO; use Model\Link; -const VERSION = 85; +const VERSION = 86; + +function version_86($pdo) +{ + $pdo->exec("ALTER TABLE custom_filters ADD COLUMN append INTEGER DEFAULT 0"); +} function version_85($pdo) { diff --git a/app/Template/custom_filter/add.php b/app/Template/custom_filter/add.php index d4e102b3..dfdae72f 100644 --- a/app/Template/custom_filter/add.php +++ b/app/Template/custom_filter/add.php @@ -13,8 +13,10 @@ form->text('filter', $values, $errors, array('required', 'maxlength="100"')) ?> user->isProjectManagementAllowed($project['id'])): ?> - form->checkbox('is_shared', t('Share with all project members'), 1) ?> + form->checkbox('is_shared', t('Share with all project members'), 1) ?> + + form->checkbox('append', t('Append filter (instead of replacement)'), 1) ?>
diff --git a/app/Template/custom_filter/edit.php b/app/Template/custom_filter/edit.php index 7525574c..f729152f 100644 --- a/app/Template/custom_filter/edit.php +++ b/app/Template/custom_filter/edit.php @@ -19,9 +19,11 @@ user->isProjectManagementAllowed($project['id'])): ?> form->checkbox('is_shared', t('Share with all project members'), 1, $values['is_shared'] == 1) ?> - form->hidden('is_shared', $values) ?> + form->hidden('is_shared', $values) ?> - + + form->checkbox('append', t('Append filter (instead of replacement)'), 1, $values['append'] == 1) ?> +
diff --git a/app/Template/custom_filter/index.php b/app/Template/custom_filter/index.php index a53d0d0a..c857e206 100644 --- a/app/Template/custom_filter/index.php +++ b/app/Template/custom_filter/index.php @@ -8,6 +8,7 @@ + @@ -22,6 +23,13 @@ + + + + + + + e($filter['owner_name'] ?: $filter['owner_username']) ?> user->getId() || $this->user->isProjectManagementAllowed($project['id'])): ?> @@ -37,4 +45,4 @@
-render('custom_filter/add', array('project' => $project, 'values' => $values, 'errors' => $errors)) ?> \ No newline at end of file +render('custom_filter/add', array('project' => $project, 'values' => $values, 'errors' => $errors)) ?> diff --git a/app/Template/project/filters.php b/app/Template/project/filters.php index c22d7b4f..eee691c5 100644 --- a/app/Template/project/filters.php +++ b/app/Template/project/filters.php @@ -68,7 +68,7 @@
-- cgit v1.2.3 From 9eed530aa8a343810235c95d39a11619d062c2ba Mon Sep 17 00:00:00 2001 From: Timo Litzbarski Date: Mon, 12 Oct 2015 14:53:13 +0000 Subject: remove trailing space --- app/Template/custom_filter/add.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app') diff --git a/app/Template/custom_filter/add.php b/app/Template/custom_filter/add.php index dfdae72f..61df148c 100644 --- a/app/Template/custom_filter/add.php +++ b/app/Template/custom_filter/add.php @@ -13,7 +13,7 @@ form->text('filter', $values, $errors, array('required', 'maxlength="100"')) ?> user->isProjectManagementAllowed($project['id'])): ?> - form->checkbox('is_shared', t('Share with all project members'), 1) ?> + form->checkbox('is_shared', t('Share with all project members'), 1) ?> form->checkbox('append', t('Append filter (instead of replacement)'), 1) ?> -- cgit v1.2.3 From 6c021baa8d9867ff2720c19e93bad4c8db0696e9 Mon Sep 17 00:00:00 2001 From: Timo Litzbarski Date: Mon, 12 Oct 2015 14:54:43 +0000 Subject: remove trailing space --- app/Template/custom_filter/edit.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app') diff --git a/app/Template/custom_filter/edit.php b/app/Template/custom_filter/edit.php index f729152f..9d296b84 100644 --- a/app/Template/custom_filter/edit.php +++ b/app/Template/custom_filter/edit.php @@ -19,7 +19,7 @@ user->isProjectManagementAllowed($project['id'])): ?> form->checkbox('is_shared', t('Share with all project members'), 1, $values['is_shared'] == 1) ?> - form->hidden('is_shared', $values) ?> + form->hidden('is_shared', $values) ?> form->checkbox('append', t('Append filter (instead of replacement)'), 1, $values['append'] == 1) ?> -- cgit v1.2.3