Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: mantisbt/mantisbt
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 27b4f79fa531
Choose a base ref
...
head repository: mantisbt/mantisbt
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: b1a1bbaeb690
Choose a head ref
  • 3 commits
  • 3 files changed
  • 2 contributors

Commits on Sep 10, 2012

  1. Hide file upload path on project edit page if storage is DB 

    The upload_file_path config option is not needed when attachments are
stored in DB, and displaying it is confusing to user.

Fixes #14700

Original patch updated to comply with coding guidelines

Signed-off-by: Damien Regad <damien.regad@merckgroup.com>
    @dregad
    JGuilbaud authored and dregad committed Sep 10, 2012

    Verified

    This commit was signed with the committer’s verified signature.
    jtojnar Jan Tojnar
    GPG key ID: 7FAB2A15F7A607A4
    Verified
    Learn about vigilant mode
    Copy the full SHA
    7aa07d8 View commit details

  2. Missing project override in project edit page 

    This is causing the config_get() calls to return the option for the
default project instead of the one being edited, which is potentially
incorrect.

Fixes #14701
    @dregad
    dregad committed Sep 10, 2012

    Verified

    This commit was signed with the committer’s verified signature.
    jtojnar Jan Tojnar
    GPG key ID: 7FAB2A15F7A607A4
    Verified
    Learn about vigilant mode
    Copy the full SHA
    e536cbc View commit details

  3. User with access to single project defaults to it, not all projects 

    Since 1.2, if a user has access to only one project, the project
selection box at top right isn't shown. This was causing issues as new
users accounts have 'All projects' as their default:

 - user can't report issues (no link for all projects)
 - categories displayed are prefixed with [project]

Until this commit, users had to define their (single) project as default
using preferences to work around the problem; this is no longer needed.

Function html_login_info() now sets the current project as appropriate
if user has only access to a single one, and also defines that as the
default.

The account preferences page has also been modified to only include 'All
projects' in the selection list when user has more than one project
available.

Fixes #9826, #14268
    @dregad
    dregad committed Sep 10, 2012

    Verified

    This commit was signed with the committer’s verified signature.
    vcunat Vladimír Čunát
    GPG key ID: E747DF1F9575A3AA
    Verified
    Learn about vigilant mode
    Copy the full SHA
    b1a1bba View commit details
Showing with 27 additions and 7 deletions.
  1. +10 −1 account_prefs_inc.php
  2. +14 −4 core/html_api.php
  3. +3 −2 manage_proj_edit_page.php
11 changes: 10 additions & 1 deletion account_prefs_inc.php
View file
Original file line number Diff line number Diff line change
@@ -75,7 +75,16 @@ function edit_account_prefs($p_user_id = null, $p_error_if_protected = true, $p_
</td>
<td width="50%">
<select name="default_project">
<?php print_project_option_list( $t_pref->default_project ) ?>
<?php
# Count number of available projects
$t_projects = current_user_get_accessible_projects();
$t_num_proj = count( $t_projects );
if( $t_num_proj == 1 ) {
$t_num_proj += count( current_user_get_accessible_subprojects( $t_projects[0] ) );
}
# Only display "All projects" in selection list if there is more than 1
print_project_option_list( $t_pref->default_project, $t_num_proj > 1 );
?>
</select>
</td>
</tr>
18 changes: 14 additions & 4 deletions core/html_api.php
View file
Original file line number Diff line number Diff line change
@@ -515,11 +515,11 @@ function html_login_info() {
echo "<span class=\"italic\">$t_now</span>";
echo '</td>';
echo '<td class="login-info-right">';
$t_show_project_selector = true;
if( count( current_user_get_accessible_projects() ) == 1 ) {

// >1
$t_project_ids = current_user_get_accessible_projects();
# Project Selector hidden if only one project visisble to user
$t_show_project_selector = true;
$t_project_ids = current_user_get_accessible_projects();
if( count( $t_project_ids ) == 1 ) {
$t_project_id = (int) $t_project_ids[0];
if( count( current_user_get_accessible_subprojects( $t_project_id ) ) == 0 ) {
$t_show_project_selector = false;
@@ -544,7 +544,17 @@ function html_login_info() {
}
echo '<input type="submit" class="button-small" value="' . lang_get( 'switch' ) . '" />';
echo '</form>';
} else {
# User has only one project, set it as both current and default
if( ALL_PROJECTS == helper_get_current_project() ) {
helper_set_current_project( $t_project_id );
current_user_set_default_project( $t_project_id );
# Force reload of current page
$t_redirect_url = str_replace( config_get( 'short_path' ), '', $_SERVER['REQUEST_URI'] );
html_meta_redirect( $t_redirect_url, 0, false );
}
}

if( OFF != config_get( 'rss_enabled' ) ) {

# Link to RSS issues feed for the selected project, including authentication details.
5 changes: 3 additions & 2 deletions manage_proj_edit_page.php
View file
Original file line number Diff line number Diff line change
@@ -36,6 +36,7 @@
$f_show_global_users = gpc_get_bool( 'show_global_users' );

project_ensure_exists( $f_project_id );
$g_project_override = $f_project_id;
access_ensure_project_level( config_get( 'manage_project_threshold' ), $f_project_id );

$row = project_get_row( $f_project_id );
@@ -115,8 +116,8 @@
</td>
</tr>

<!-- File upload path (if uploading is enabled) -->
<?php if ( file_is_uploading_enabled() ) { ?>
<!-- File upload path (if uploading is enabled and uploading to disk) -->
<?php if ( file_is_uploading_enabled() && DATABASE !== config_get( 'file_upload_method' ) ) { ?>
<tr <?php echo helper_alternate_class() ?>>
<td class="category">
<?php echo lang_get( 'upload_file_path' ) ?>