diff --git a/AUTHORS b/AUTHORS index 10480b8..1a1790b 100644 --- a/AUTHORS +++ b/AUTHORS @@ -31,6 +31,7 @@ Much appreciated contributors (in alphabetical order): Raphaël Emourgeon Samuel Suther - German translation Sindre R. Myren + Stewart Platt Thomas Keller - Monotone support Vladimir Solomatin William Martin diff --git a/NEWS.mdtext b/NEWS.mdtext index e16f211..c3bbfda 100644 --- a/NEWS.mdtext +++ b/NEWS.mdtext @@ -5,6 +5,7 @@ ## Bugfixes - monotone zip archive entries now all carry the revision date as mtime (issue 645) +- Timeline only displays filter options for items a user has actually access to (issue 655) ## Documentation diff --git a/src/IDF/Views/Project.php b/src/IDF/Views/Project.php index 2befafc..528a2dd 100644 --- a/src/IDF/Views/Project.php +++ b/src/IDF/Views/Project.php @@ -62,20 +62,26 @@ class IDF_Views_Project } /** - * Returns an associative array with available model filters + * Returns an associative array with all accessible model filters * * @return array */ - private static function getAvailableModelFilters() + private static function getAccessibleModelFilters($request) { - return array( - 'all' => __('All Updates'), - 'commits' => __('Commits'), - 'issues' => __('Issues and Comments'), - 'downloads' => __('Downloads'), - 'documents' => __('Documents'), - 'reviews' => __('Reviews and Patches'), - ); + $filters = array('all' => __('All Updates')); + + if (true === IDF_Precondition::accessSource($request)) + $filters['commits'] = __('Commits'); + if (true === IDF_Precondition::accessIssues($request)) + $filters['issues'] = __('Issues and Comments'); + if (true === IDF_Precondition::accessDownloads($request)) + $filters['downloads'] = __('Downloads'); + if (true === IDF_Precondition::accessWiki($request)) + $filters['documents'] = __('Documents'); + if (true === IDF_Precondition::accessReview($request)) + $filters['reviews'] = __('Reviews and Patches'); + + return $filters; } /** @@ -141,11 +147,11 @@ class IDF_Views_Project $prj = $request->project; $model_filter = @$match[2]; - $all_model_filters = self::getAvailableModelFilters(); - if (!array_key_exists($model_filter, $all_model_filters)) { + $accessible_model_filters = self::getAccessibleModelFilters($request); + if (!array_key_exists($model_filter, $accessible_model_filters)) { $model_filter = 'all'; } - $title = (string)$prj . ' ' . $all_model_filters[$model_filter]; + $title = (string)$prj . ' ' . $accessible_model_filters[$model_filter]; $pag = new IDF_Timeline_Paginator(new IDF_Timeline()); $pag->class = 'recent-issues'; @@ -183,7 +189,7 @@ class IDF_Views_Project 'feedurl' => $feedurl, 'timeline' => $pag, 'model_filter' => $model_filter, - 'all_model_filters' => $all_model_filters, + 'accessible_model_filters' => $accessible_model_filters, ), $request); @@ -214,11 +220,11 @@ class IDF_Views_Project $prj = $request->project; $model_filter = @$match[2]; - $all_model_filters = self::getAvailableModelFilters(); - if (!array_key_exists($model_filter, $all_model_filters)) { + $accessible_model_filters = self::getAccessibleModelFilters($request); + if (!array_key_exists($model_filter, $accessible_model_filters)) { $model_filter = 'all'; } - $title = $all_model_filters[$model_filter]; + $title = $accessible_model_filters[$model_filter]; $classes = self::determineModelClasses($request, $model_filter); $sqls = sprintf('model_class IN (%s)', implode(', ', $classes)); diff --git a/src/IDF/templates/idf/project/timeline.html b/src/IDF/templates/idf/project/timeline.html index 55186ac..92e66d3 100644 --- a/src/IDF/templates/idf/project/timeline.html +++ b/src/IDF/templates/idf/project/timeline.html @@ -15,7 +15,7 @@ {block context}

{trans 'All Updates'}

{trans 'Filter by type'}
-{foreach $all_model_filters as $filter_key => $filter_name} +{foreach $accessible_model_filters as $filter_key => $filter_name} {if $filter_key != 'all'} {$filter_name}
{/if} diff --git a/test/IDF/DiffTest.php b/test/IDF/DiffTest.php index 68e4ca3..c293768 100644 --- a/test/IDF/DiffTest.php +++ b/test/IDF/DiffTest.php @@ -1,4 +1,25 @@