← Back to branch summary

~launchpad-pqm/launchpad/devel

« back to all changes in this revision

Viewing changes to lib/lp/code/browser/branchlisting.py

[r=gmb][bug=827935] Add a new simplified menu branch without the
        counts.

Show diffs side-by-side

added added

removed removed

Lines of Context:
lib/lp/code/browser/branchlisting.py
31
31
    ]
32
32
 
33
33
from operator import attrgetter
 
34
import urlparse
34
35
 
35
36
from lazr.delegates import delegates
36
37
from lazr.enum import (
41
42
    Asc,
42
43
    Desc,
43
44
    )
44
 
import urlparse
45
45
from z3c.ptcompat import ViewPageTemplateFile
46
46
from zope.component import getUtility
47
47
from zope.formlib import form
135
135
from lp.registry.interfaces.sourcepackage import ISourcePackageFactory
136
136
from lp.registry.model.sourcepackage import SourcePackage
137
137
from lp.services.browser_helpers import get_plural_text
 
138
from lp.services.features import getFeatureFlag
138
139
from lp.services.propertycache import cachedproperty
139
140
 
140
141
 
850
851
    usedfor = IPerson
851
852
    facet = 'branches'
852
853
    links = ['registered', 'owned', 'subscribed', 'addbranch',
853
 
             'active_reviews', 'mergequeues']
 
854
             'active_reviews', 'mergequeues',
 
855
             'simplified_subscribed', 'simplified_registered',
 
856
             'simplified_owned', 'simplified_active_reviews']
854
857
    extra_attributes = [
855
858
        'active_review_count',
856
859
        'owned_branch_count',
858
861
        'show_summary',
859
862
        'subscribed_branch_count',
860
863
        'mergequeue_count',
 
864
        'simplified_branches_menu',
861
865
        ]
862
866
 
863
867
    def _getCountCollection(self):
881
885
        """
882
886
        return self.context
883
887
 
 
888
    @cachedproperty
 
889
    def has_branches(self):
 
890
        """Should the template show the summary view with the links."""
 
891
 
884
892
    @property
885
893
    def show_summary(self):
886
894
        """Should the template show the summary view with the links."""
887
 
        return (self.owned_branch_count or
 
895
 
 
896
        if self.simplified_branches_menu:
 
897
            return (
 
898
                self.registered_branches_not_empty or
 
899
                self.owned_branches_not_empty or
 
900
                self.subscribed_branches_not_empty or
 
901
                self.active_reviews_not_empty
 
902
                )
 
903
        else:
 
904
            return (
 
905
                self.owned_branch_count or
888
906
                self.registered_branch_count or
889
907
                self.subscribed_branch_count or
890
 
                self.active_review_count)
 
908
                self.active_review_count
 
909
                )
 
910
 
 
911
    @cachedproperty
 
912
    def simplified_branches_menu(self):
 
913
        return getFeatureFlag('code.simplified_branches_menu.enabled')
 
914
 
 
915
    @cachedproperty
 
916
    def registered_branches_not_empty(self):
 
917
        """False if the number of branches registered by self.person
 
918
        is zero.
 
919
        """
 
920
        return (
 
921
            not self._getCountCollection().registeredBy(
 
922
                self.person).is_empty())
 
923
 
 
924
    @cachedproperty
 
925
    def owned_branches_not_empty(self):
 
926
        """False if the number of branches owned by self.person is zero."""
 
927
        return not self._getCountCollection().ownedBy(self.person).is_empty()
 
928
 
 
929
    @cachedproperty
 
930
    def subscribed_branches_not_empty(self):
 
931
        """False if the number of branches subscribed to by self.person
 
932
        is zero.
 
933
        """
 
934
        return (
 
935
            not self._getCountCollection().subscribedBy(
 
936
                self.person).is_empty())
 
937
 
 
938
    @cachedproperty
 
939
    def active_reviews_not_empty(self):
 
940
        """Return the number of active reviews for self.person's branches."""
 
941
        active_reviews = PersonActiveReviewsView(self.context, self.request)
 
942
        return not active_reviews.getProposals().is_empty()
 
943
 
 
944
    def simplified_owned(self):
 
945
        return Link(
 
946
            canonical_url(self.context, rootsite='code'),
 
947
            'Owned branches',
 
948
            enabled=self.owned_branches_not_empty)
 
949
 
 
950
    def simplified_registered(self):
 
951
        person_is_individual = (not self.person.is_team)
 
952
        return Link(
 
953
            '+registeredbranches',
 
954
            'Registered branches',
 
955
            enabled=(
 
956
                person_is_individual and
 
957
                self.registered_branches_not_empty))
 
958
 
 
959
    def simplified_subscribed(self):
 
960
        return Link(
 
961
            '+subscribedbranches',
 
962
            'Subscribed branches',
 
963
            enabled=self.subscribed_branches_not_empty)
 
964
 
 
965
    def simplified_active_reviews(self):
 
966
        return Link(
 
967
            '+activereviews',
 
968
            'Active reviews',
 
969
            enabled=self.active_reviews_not_empty)
891
970
 
892
971
    @cachedproperty
893
972
    def registered_branch_count(self):