Property changes on: . ___________________________________________________________________ Name: svn:ignore - !core custom_constant_inc.php custom_strings_inc.php custom_functions_inc.php config_inc.php .#* .project project.index .DS_Store custom_relationships_inc.php + !core custom_constant_inc.php custom_strings_inc.php custom_functions_inc.php config_inc.php .#* .project project.index .DS_Store custom_relationships_inc.php jpgraph avatar Index: query_store_page.php =================================================================== --- query_store_page.php (revision 5305) +++ query_store_page.php (working copy) @@ -40,6 +40,11 @@
-
+
'; + print ' '; print lang_get( 'make_public' ); print '
'; } ?> - > + + >

+ + + +
-
- -
+ +
+ +
+ +
+ + + + + +
+ '; html_page_bottom1( __FILE__ ); Index: view_filters_page.php =================================================================== --- view_filters_page.php (revision 5305) +++ view_filters_page.php (working copy) @@ -97,6 +97,7 @@ $t_filter_cols = config_get( 'filter_custom_fields_per_row' ); $t_custom_cols = 1; $t_custom_rows = 0; + $f_source_query_id = gpc_get_string( 'source_query_id' , null); #get valid target fields $t_fields = helper_get_columns_to_view(); @@ -180,7 +181,11 @@ + + + + - + \ No newline at end of file Index: view_all_set.php =================================================================== --- view_all_set.php (revision 5305) +++ view_all_set.php (working copy) @@ -28,6 +28,7 @@ $f_source_query_id = gpc_get_int( 'source_query_id', -1 ); $f_print = gpc_get_bool( 'print' ); $f_temp_filter = gpc_get_bool( 'temporary' ); + $f_edit = gpc_get_bool( 'edit' ); # validate filter type $f_default_view_type = 'simple'; @@ -519,11 +520,13 @@ gpc_set_cookie( config_get( 'view_all_cookie' ), $t_row_id, time()+config_get( 'cookie_time_length' ), config_get( 'cookie_path' ) ); } - # redirect to print_all or view_all page - if ( $f_print ) { - $t_redirect_url = 'print_all_bug_page.php'; + # redirect to print_all, view_all page or filter_edit + if ($f_print && !$f_edit) { + $t_redirect_url = 'print_all_bug_page.php'; + } else if (!$f_edit){ + $t_redirect_url = 'view_all_bug_page.php'; } else { - $t_redirect_url = 'view_all_bug_page.php'; + $t_redirect_url = 'filter_edit_page.php?source_query_id=' . $f_source_query_id; } if ( $f_temp_filter ) { @@ -533,4 +536,4 @@ } else { print_header_redirect( $t_redirect_url ); } -?> +?> \ No newline at end of file Index: query_store.php =================================================================== --- query_store.php (revision 5305) +++ query_store.php (working copy) @@ -36,27 +36,35 @@ $f_query_name = strip_tags( gpc_get_string( 'query_name' ) ); $f_is_public = gpc_get_bool( 'is_public' ); $f_all_projects = gpc_get_bool( 'all_projects' ); + $f_edit = gpc_get_bool( 'edit', false ); + $f_source_query_id = gpc_get_string( 'source_query_id', null); - $t_query_redirect_url = 'query_store_page.php'; + $t_query_redirect_url = 'query_store_page.php?'; + if (!is_blank($f_source_query_id)) { + $t_query_redirect_url = $t_query_redirect_url . 'edit=true&source_query_id=' . $f_source_query_id . '&'; + } + # We can't have a blank name if ( is_blank( $f_query_name ) ) { - $t_query_redirect_url = $t_query_redirect_url . '?error_msg=' + $t_query_redirect_url = $t_query_redirect_url . 'error_msg=' . urlencode( lang_get( 'query_blank_name' ) ); print_header_redirect( $t_query_redirect_url ); } # Check and make sure they don't already have a # query with the same name - $t_query_arr = filter_db_get_available_queries(); - foreach( $t_query_arr as $t_id => $t_name ) { - if ( $f_query_name == $t_name ) { - $t_query_redirect_url = $t_query_redirect_url . '?error_msg=' - . urlencode( lang_get( 'query_dupe_name' ) ); - print_header_redirect( $t_query_redirect_url ); - exit; + if (is_null($f_source_query_id)) { + $t_query_arr = filter_db_get_available_queries(); + foreach( $t_query_arr as $t_id => $t_name ) { + if ( $f_query_name == $t_name ) { + $t_query_redirect_url = $t_query_redirect_url . 'error_msg=' + . urlencode( lang_get( 'query_dupe_name' ) ); + print_header_redirect( $t_query_redirect_url ); + exit; + } } - } + } $t_project_id = helper_get_current_project(); if ( $f_all_projects ) { @@ -66,13 +74,17 @@ $t_filter_string = filter_db_get_filter( gpc_get_cookie( config_get( 'view_all_cookie' ), '' ) ); $t_new_row_id = filter_db_set_for_current_user($t_project_id, $f_is_public, - $f_query_name, $t_filter_string); + $f_query_name, $t_filter_string, $f_source_query_id); if ( $t_new_row_id == -1 ) { $t_query_redirect_url = $t_query_redirect_url . '?error_msg=' . urlencode( lang_get( 'query_store_error' ) ); print_header_redirect( $t_query_redirect_url ); } else { - print_header_redirect( 'view_all_bug_page.php' ); + if (!$f_edit) { + print_header_redirect( 'view_all_bug_page.php' ); + } else { + print_header_redirect( 'filter_edit_page.php?edit=true&source_query_id=' . $f_source_query_id ); + } } ?> Index: view_all_bug_page.php =================================================================== --- view_all_bug_page.php (revision 5305) +++ view_all_bug_page.php (working copy) @@ -73,4 +73,4 @@ include( dirname( __FILE__ ) . DIRECTORY_SEPARATOR . 'view_all_inc.php' ); html_page_bottom1( __FILE__ ); -?> +?> \ No newline at end of file Index: filter_edit_page.php =================================================================== --- filter_edit_page.php (revision 0) +++ filter_edit_page.php (revision 0) @@ -0,0 +1,90 @@ +. + + # -------------------------------------------------------- + # $Revision: 1.62.2.1 $ + # $Author: giallu $ + # $Date: 2007-10-13 22:34:48 $ + # + # $Id: view_all_bug_page.php,v 1.62.2.1 2007-10-13 22:34:48 giallu Exp $ + # -------------------------------------------------------- +?> + + + Index: query_view_page.php =================================================================== --- query_view_page.php (revision 5305) +++ query_view_page.php (working copy) @@ -45,53 +45,96 @@ $t_use_query_url = 'view_all_set.php?type=3&source_query_id='; $t_delete_query_url = 'query_delete_page.php?source_query_id='; + $t_edit_query_url = 'view_all_set.php?type=3&edit=true&advanced_view_type=true&source_query_id='; $t_rss_enabled = config_get( 'rss_enabled' ); ?>
+ + + + + + + + + + + $t_name ) { - if ( $t_column_count == 0 ) { - print ''; - } + $filter_row = filter_get_row( $t_id ); + print ''; + print ''; - print '' . string_display( $t_name ) . ''; + print ''; + print ''; + + print ''; + + print ''; + + print ''; + + print ''; + + print ''; - $t_column_count++; - if ( $t_column_count == $t_max_column_count ) { - print ''; - $t_column_count = 0; - } - } + print ''; - # Tidy up this row - if ( ( $t_column_count > 0 ) && ( $t_column_count < $t_max_column_count ) ) { - for ( $i = $t_column_count; $i < $t_max_column_count; $i++ ) { - print ''; - } print ''; } ?>
+ + + + + + + + + + + +
'; - if ( OFF != $t_rss_enabled ) { # Use the "new" RSS link style. print_rss( rss_get_issues_feed_url( null, null, $t_id ), lang_get( 'rss' ) ); echo ' '; } + print ''; + print string_display( $filter_row['id'] ); + print ''; + print string_display( $filter_row['name'] ); + print ''; + print '' . string_display( lang_get_defaulted($t_name) ) . ''; + print ''; + print '' . string_display( user_get_name($filter_row['user_id']) ) . ''; + print ''; + if ($filter_row['project_id'] == ALL_PROJECTS) { + print string_display( lang_get_defaulted(all_projects) ); + } else { + print string_display( project_get_field($filter_row['project_id'], 'name')); + } + print ''; + print ''; + print ''; if ( filter_db_can_delete_filter( $t_id ) ) { echo ' '; print_button( $t_delete_query_url . db_prepare_int( $t_id ), lang_get( 'delete_query' ) ); } - print '
'; + print_button( $t_edit_query_url . db_prepare_int( $t_id ), lang_get( 'edit_query' ) ); + print ' 
- + \ No newline at end of file Index: set_project.php =================================================================== --- set_project.php (revision 5305) +++ set_project.php (working copy) @@ -63,13 +63,15 @@ if ( strtolower( $t_path ) == strtolower( substr( $_SERVER['HTTP_REFERER'], 0, strlen( $t_path ) ) ) ) { $t_referrer_page = substr( $_SERVER['HTTP_REFERER'], strlen( $t_path ) ); # if view_all_bug_page, pass on filter - if ( eregi( 'view_all_bug_page.php', $t_referrer_page ) ) { + if ( eregi( 'view_all_bug_page.php', $t_referrer_page ) || eregi( 'filter_edit_page.php', $t_referrer_page ) ) { $t_source_filter_id = filter_db_get_project_current( $f_project_id ); $t_redirect_url = 'view_all_set.php?type=4'; if ( $t_source_filter_id !== null ) { $t_redirect_url = 'view_all_set.php?type=3&source_query_id=' . $t_source_filter_id; } + } else if ( eregi( 'query_store_page.php', $t_referrer_page ) ) { + $t_redirect_url = $t_home_page; } else if ( eregi( '_page.php', $t_referrer_page ) ) { # get just the page component if ( strpos( $t_referrer_page, '?' ) !== FALSE ) { Index: core/filter_api.php =================================================================== --- core/filter_api.php (revision 5305) +++ core/filter_api.php (working copy) @@ -1727,7 +1727,13 @@ } } - $t_filters_url = 'view_filters_page.php?for_screen=' . $p_for_screen; + $t_filters_url = 'view_filters_page.php?'; + $f_source_query_id = gpc_get_string('source_query_id', null); + if (!is_null($f_source_query_id)) { + $t_filters_url = $t_filters_url . 'source_query_id=' . $f_source_query_id . '&'; + } + $t_filters_url = $t_filters_url . 'for_screen=' . $p_for_screen; + if ( 'advanced' == $t_view_type ) { $t_filters_url = $t_filters_url . '&view_type=advanced'; } @@ -1742,6 +1748,16 @@ } ?> + + + + + + + > : @@ -2874,10 +2890,18 @@
+ 0) { + ?> +
+ + +
+ - @@ -2888,7 +2912,7 @@ # Add a filter to the database for the current user function filter_db_set_for_current_user( $p_project_id, $p_is_public, - $p_name, $p_filter_string ) { + $p_name, $p_filter_string, $p_source_query_id = null ) { $t_user_id = auth_get_current_user_id(); $c_project_id = db_prepare_int( $p_project_id ); $c_is_public = db_prepare_bool( $p_is_public, false ); @@ -2912,6 +2936,9 @@ WHERE user_id=" . db_param(0) . " AND project_id=" . db_param(1) . " AND name=" . db_param(2); + if (!is_null($p_source_query_id)) { + $query = $query . " OR id=$p_source_query_id"; + } $result = db_query_bound( $query, Array( $t_user_id, $c_project_id, $c_name ) ); if ( db_num_rows( $result ) > 0 ) { @@ -2919,8 +2946,10 @@ $query = "UPDATE $t_filters_table SET is_public=" . db_param(0) . ", - filter_string=" . db_param(1) . " - WHERE id=" . db_param(2); + filter_string=" . db_param(1) . ", + project_id=$c_project_id, + name='$c_name' + WHERE id=" . db_param(2); db_query_bound( $query, Array( $c_is_public, $p_filter_string, $row['id'] ) ); return $row['id']; Index: css/default.css =================================================================== --- css/default.css (revision 5305) +++ css/default.css (working copy) @@ -63,6 +63,7 @@ td.print-category { font-size: 8pt; color: #000000; font-weight: bold; text-align: right; padding: 2px; } td.print-bottom { border-bottom: 1px solid #000000; } td.print-spacer { background-color: #ffffff; color: #000000; font-size: 1pt; line-height: 0.1; padding: 0px;} +td.filter-caption { font-size: 8pt; font-weight: bold;} tr {} tr.spacer { background-color: #ffffff; color: #000000; height: 5px; } Index: filter_edit_page.php =================================================================== --- filter_edit_page.php (revision 0) +++ filter_edit_page.php (revision 0) @@ -0,0 +1,90 @@ +. + + # -------------------------------------------------------- + # $Revision: 1.62.2.1 $ + # $Author: giallu $ + # $Date: 2007-10-13 22:34:48 $ + # + # $Id: view_all_bug_page.php,v 1.62.2.1 2007-10-13 22:34:48 giallu Exp $ + # -------------------------------------------------------- +?> + + + Index: lang/strings_english.txt =================================================================== --- lang/strings_english.txt (revision 5305) +++ lang/strings_english.txt (working copy) @@ -1190,15 +1190,19 @@ $s_use_query = 'Use Filter'; $s_delete_query = 'Delete Filter'; $s_query_deleted = 'Filter Deleted'; +$s_edit_query = 'Edit Filter'; $s_save_query = 'Save Current Filter'; +$s_update_query = 'Update Current Filter'; $s_reset_query = 'Reset Filter'; $s_query_name = 'Filter Name'; +$s_query_name_translated = 'Filter Name (translated)'; $s_query_exists = 'This particular query appears to already exist.'; $s_query_dupe_name = 'Another query already has this name. Please choose a different name for this query.'; $s_query_blank_name = 'You cannot store a filter without a name. Please give this filter a name before saving it.'; $s_query_store_error = 'There was an error saving this query.'; $s_open_queries = 'Manage Filters'; $s_query_delete_msg = 'Are you sure you wish to delete this query?'; +$s_query_project = 'Project'; # view_bug_advanced_page.php $s_view_simple_link = 'View Simple'; Index: lang/strings_german.txt =================================================================== --- lang/strings_german.txt (revision 5305) +++ lang/strings_german.txt (working copy) @@ -1177,12 +1177,14 @@ $s_save_query = 'Filter speichern'; $s_reset_query = 'Filter zurücksetzen'; $s_query_name = 'Filtername'; +$s_query_name_translated = 'Filtername (übersetzt)'; $s_query_exists = 'Ein Filter mit diesen Optionen existiert bereits.'; $s_query_dupe_name = 'Ein Filter mit diesem Namen existiert bereits. Bitte geben Sie einen anderen Namen ein.'; $s_query_blank_name = 'Bitte geben Sie einen Namen für diesen Filter ein.'; $s_query_store_error = 'Ein Fehler ist beim Speichern des Filters aufgetreten.'; $s_open_queries = 'Filter verwalten'; $s_query_delete_msg = 'Möchten Sie diesen Filter wirklich löschen?'; +$s_query_project = 'Projekt'; # view_bug_advanced_page.php $s_view_simple_link = 'einfache Anzeige'; Index: query_store.php =================================================================== --- query_store.php (revision 5305) +++ query_store.php (working copy) @@ -36,27 +36,35 @@ $f_query_name = strip_tags( gpc_get_string( 'query_name' ) ); $f_is_public = gpc_get_bool( 'is_public' ); $f_all_projects = gpc_get_bool( 'all_projects' ); + $f_edit = gpc_get_bool( 'edit', false ); + $f_source_query_id = gpc_get_string( 'source_query_id', null); - $t_query_redirect_url = 'query_store_page.php'; + $t_query_redirect_url = 'query_store_page.php?'; + if (!is_blank($f_source_query_id)) { + $t_query_redirect_url = $t_query_redirect_url . 'edit=true&source_query_id=' . $f_source_query_id . '&'; + } + # We can't have a blank name if ( is_blank( $f_query_name ) ) { - $t_query_redirect_url = $t_query_redirect_url . '?error_msg=' + $t_query_redirect_url = $t_query_redirect_url . 'error_msg=' . urlencode( lang_get( 'query_blank_name' ) ); print_header_redirect( $t_query_redirect_url ); } # Check and make sure they don't already have a # query with the same name - $t_query_arr = filter_db_get_available_queries(); - foreach( $t_query_arr as $t_id => $t_name ) { - if ( $f_query_name == $t_name ) { - $t_query_redirect_url = $t_query_redirect_url . '?error_msg=' - . urlencode( lang_get( 'query_dupe_name' ) ); - print_header_redirect( $t_query_redirect_url ); - exit; + if (is_null($f_source_query_id)) { + $t_query_arr = filter_db_get_available_queries(); + foreach( $t_query_arr as $t_id => $t_name ) { + if ( $f_query_name == $t_name ) { + $t_query_redirect_url = $t_query_redirect_url . 'error_msg=' + . urlencode( lang_get( 'query_dupe_name' ) ); + print_header_redirect( $t_query_redirect_url ); + exit; + } } - } + } $t_project_id = helper_get_current_project(); if ( $f_all_projects ) { @@ -66,13 +74,17 @@ $t_filter_string = filter_db_get_filter( gpc_get_cookie( config_get( 'view_all_cookie' ), '' ) ); $t_new_row_id = filter_db_set_for_current_user($t_project_id, $f_is_public, - $f_query_name, $t_filter_string); + $f_query_name, $t_filter_string, $f_source_query_id); if ( $t_new_row_id == -1 ) { $t_query_redirect_url = $t_query_redirect_url . '?error_msg=' . urlencode( lang_get( 'query_store_error' ) ); print_header_redirect( $t_query_redirect_url ); } else { - print_header_redirect( 'view_all_bug_page.php' ); + if (!$f_edit) { + print_header_redirect( 'view_all_bug_page.php' ); + } else { + print_header_redirect( 'filter_edit_page.php?edit=true&source_query_id=' . $f_source_query_id ); + } } ?> Index: query_store_page.php =================================================================== --- query_store_page.php (revision 5305) +++ query_store_page.php (working copy) @@ -40,6 +40,11 @@
-
+
'; + print ' '; print lang_get( 'make_public' ); print '
'; } ?> - > + + >

+ + + +
-
- -
+ +
+ +
+ +
+ + + + + +
+ '; html_page_bottom1( __FILE__ ); Index: query_view_page.php =================================================================== --- query_view_page.php (revision 5305) +++ query_view_page.php (working copy) @@ -45,53 +45,96 @@ $t_use_query_url = 'view_all_set.php?type=3&source_query_id='; $t_delete_query_url = 'query_delete_page.php?source_query_id='; + $t_edit_query_url = 'view_all_set.php?type=3&edit=true&advanced_view_type=true&source_query_id='; $t_rss_enabled = config_get( 'rss_enabled' ); ?>
+ + + + + + + + + + + $t_name ) { - if ( $t_column_count == 0 ) { - print ''; - } + $filter_row = filter_get_row( $t_id ); + print ''; + print ''; - print '' . string_display( $t_name ) . ''; + print ''; + print ''; + + print ''; + + print ''; + + print ''; + + print ''; + + print ''; - $t_column_count++; - if ( $t_column_count == $t_max_column_count ) { - print ''; - $t_column_count = 0; - } - } + print ''; - # Tidy up this row - if ( ( $t_column_count > 0 ) && ( $t_column_count < $t_max_column_count ) ) { - for ( $i = $t_column_count; $i < $t_max_column_count; $i++ ) { - print ''; - } print ''; } ?>
+ + + + + + + + + + + +
'; - if ( OFF != $t_rss_enabled ) { # Use the "new" RSS link style. print_rss( rss_get_issues_feed_url( null, null, $t_id ), lang_get( 'rss' ) ); echo ' '; } + print ''; + print string_display( $filter_row['id'] ); + print ''; + print string_display( $filter_row['name'] ); + print ''; + print '' . string_display( lang_get_defaulted($t_name) ) . ''; + print ''; + print '' . string_display( user_get_name($filter_row['user_id']) ) . ''; + print ''; + if ($filter_row['project_id'] == ALL_PROJECTS) { + print string_display( lang_get_defaulted(all_projects) ); + } else { + print string_display( project_get_field($filter_row['project_id'], 'name')); + } + print ''; + print ''; + print ''; if ( filter_db_can_delete_filter( $t_id ) ) { echo ' '; print_button( $t_delete_query_url . db_prepare_int( $t_id ), lang_get( 'delete_query' ) ); } - print '
'; + print_button( $t_edit_query_url . db_prepare_int( $t_id ), lang_get( 'edit_query' ) ); + print ' 
- + \ No newline at end of file Index: set_project.php =================================================================== --- set_project.php (revision 5305) +++ set_project.php (working copy) @@ -63,13 +63,15 @@ if ( strtolower( $t_path ) == strtolower( substr( $_SERVER['HTTP_REFERER'], 0, strlen( $t_path ) ) ) ) { $t_referrer_page = substr( $_SERVER['HTTP_REFERER'], strlen( $t_path ) ); # if view_all_bug_page, pass on filter - if ( eregi( 'view_all_bug_page.php', $t_referrer_page ) ) { + if ( eregi( 'view_all_bug_page.php', $t_referrer_page ) || eregi( 'filter_edit_page.php', $t_referrer_page ) ) { $t_source_filter_id = filter_db_get_project_current( $f_project_id ); $t_redirect_url = 'view_all_set.php?type=4'; if ( $t_source_filter_id !== null ) { $t_redirect_url = 'view_all_set.php?type=3&source_query_id=' . $t_source_filter_id; } + } else if ( eregi( 'query_store_page.php', $t_referrer_page ) ) { + $t_redirect_url = $t_home_page; } else if ( eregi( '_page.php', $t_referrer_page ) ) { # get just the page component if ( strpos( $t_referrer_page, '?' ) !== FALSE ) { Index: view_all_bug_page.php =================================================================== --- view_all_bug_page.php (revision 5305) +++ view_all_bug_page.php (working copy) @@ -73,4 +73,4 @@ include( dirname( __FILE__ ) . DIRECTORY_SEPARATOR . 'view_all_inc.php' ); html_page_bottom1( __FILE__ ); -?> +?> \ No newline at end of file Index: view_all_set.php =================================================================== --- view_all_set.php (revision 5305) +++ view_all_set.php (working copy) @@ -28,6 +28,7 @@ $f_source_query_id = gpc_get_int( 'source_query_id', -1 ); $f_print = gpc_get_bool( 'print' ); $f_temp_filter = gpc_get_bool( 'temporary' ); + $f_edit = gpc_get_bool( 'edit' ); # validate filter type $f_default_view_type = 'simple'; @@ -519,11 +520,13 @@ gpc_set_cookie( config_get( 'view_all_cookie' ), $t_row_id, time()+config_get( 'cookie_time_length' ), config_get( 'cookie_path' ) ); } - # redirect to print_all or view_all page - if ( $f_print ) { - $t_redirect_url = 'print_all_bug_page.php'; + # redirect to print_all, view_all page or filter_edit + if ($f_print && !$f_edit) { + $t_redirect_url = 'print_all_bug_page.php'; + } else if (!$f_edit){ + $t_redirect_url = 'view_all_bug_page.php'; } else { - $t_redirect_url = 'view_all_bug_page.php'; + $t_redirect_url = 'filter_edit_page.php?source_query_id=' . $f_source_query_id; } if ( $f_temp_filter ) { @@ -533,4 +536,4 @@ } else { print_header_redirect( $t_redirect_url ); } -?> +?> \ No newline at end of file Index: view_filters_page.php =================================================================== --- view_filters_page.php (revision 5305) +++ view_filters_page.php (working copy) @@ -97,6 +97,7 @@ $t_filter_cols = config_get( 'filter_custom_fields_per_row' ); $t_custom_cols = 1; $t_custom_rows = 0; + $f_source_query_id = gpc_get_string( 'source_query_id' , null); #get valid target fields $t_fields = helper_get_columns_to_view(); @@ -180,7 +181,11 @@ + + + + - + \ No newline at end of file