senaite.core.browser.widgets.analysisprofileswidget.AnalysisProfilesWidget.folderitem() - Code Metrics - Inspection of "Migrate Analysis Profiles to Dexterity (#2492)" - senaite/senaite.core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — 2.x ( 864250...eaa7c8 )

by Jordi

created 2024-02-18 12:55 UTC

AnalysisProfilesWidget.folderitem() C

↳ Parent: senaite.core.browser.widgets.analysisprofileswidget

Complexity

Conditions

Size

Total Lines	71
Code Lines	45

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	45
dl	0
loc	71
rs	5.9999
c	0
b	0
f	0
cc	10
nop	4

How to fix Long Method Complexity

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

If many parameters/temporary variables are present:
- Replace temporary variables with Query
- Introduce parameter object; often combined with preserve whole object
- If the above two are insufficient: Replace method with method object
If you have long conditionals: Decompose Conditional
Otherwise: Extract method

Complexity

Complex classes like senaite.core.browser.widgets.analysisprofileswidget.AnalysisProfilesWidget.folderitem() often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

# -*- coding: utf-8 -*-

import collections

from bika.lims import api
from bika.lims import bikaMessageFactory as _
from bika.lims.api.security import check_permission
from bika.lims.utils import format_supsub
from bika.lims.utils import get_image
from bika.lims.utils import get_link
from plone.memoize import view
from senaite.core.catalog import SETUP_CATALOG
from senaite.core.permissions import FieldEditProfiles
from senaite.core.z3cform.widgets.listing.view import DefaultListingWidget
from zope.i18n.locales import locales


class AnalysisProfilesWidget(DefaultListingWidget):
    """Listing widget for Analysis Profiles
    """
    def __init__(self, field, request):
        super(AnalysisProfilesWidget, self).__init__(field, request)

        self.catalog = SETUP_CATALOG
        self.contentFilter = {
            "portal_type": "AnalysisService",
            "sort_on": "sortable_title",
            "sort_order": "ascending",
            "is_active": True,
        }

        # group the current records by UID
        self.records = {}
        for record in self.get_value():
            uid = record.get("uid")
            self.records[uid] = record

        # listing config
        self.allow_edit = True
        self.context_actions = {}
        self.fetch_transitions_on_select = False
        self.omit_form = True
        self.pagesize = 999999
        self.show_column_toggles = False
        self.show_search = True
        self.show_select_all_checkbox = False
        self.show_select_column = True
        self.show_table_footer = False

        # Categories
        if self.show_categories_enabled():
            self.categories = []
            self.show_categories = True
            self.expand_all_categories = False

        self.columns = collections.OrderedDict((
            ("Title", {
                "title": _(
                    u"listing_analysisprofileswidget_column_title",
                    default=u"Service"
                ),
                "index": "sortable_title",
                "sortable": False
            }),
            ("Keyword", {
                "title": _(
                    u"listing_analysisprofileswidget_column_keyword",
                    default=u"Keyword"
                ),
                "sortable": False
            }),
            ("Methods", {
                "title": _(
                    u"listing_analysisprofileswidget_column_methods",
                    default=u"Methods"
                ),
                "sortable": False
            }),
            ("Unit", {
                "title": _(
                    u"listing_analysisprofileswidget_column_unit",
                    default=u"Unit"
                ),
                "sortable": False
            }),
            ("Price", {
                "title": _(
                    u"listing_analysisprofileswidget_column_price",
                    default=u"Price"
                ),
                "sortable": False,
            }),
            ("Hidden", {
                "title": _(
                    u"listing_analysisprofileswidget_column_hidden",
                    default=u"Hidden"
                ),
                "sortable": False,
            }),
        ))

        cols = self.columns.keys()
        if not self.show_prices():
            cols.remove("Price")

        self.review_states = [
            {
                "id": "default",
                "title": _(
                    u"listing_analysisprofileswidget_state_all",
                    default=u"All"
                ),
                "contentFilter": {},
                "transitions": [{"id": "disallow-all-possible-transitions"}],
                "columns": cols,
            },
        ]

    @view.memoize
    def show_categories_enabled(self):
        """Check in the setup if categories are enabled
        """
        return self.context.bika_setup.getCategoriseAnalysisServices()

    @view.memoize
    def show_prices(self):
        """Checks if prices should be shown or not
        """
        setup = api.get_setup()
        return setup.getShowPrices()

    @view.memoize
    def get_currency_symbol(self):
        """Get the currency Symbol
        """
        locale = locales.getLocale("en")
        setup = api.get_setup()
        currency = setup.getCurrency()
        return locale.numbers.currencies[currency].symbol

    @view.memoize
    def get_decimal_mark(self):
        """Returns the decimal mark
        """
        setup = api.get_setup()
        return setup.getDecimalMark()

    @view.memoize
    def format_price(self, price):
        """Formats the price with the set decimal mark and correct currency
        """
        return u"{} {}{}{:02d}".format(
            self.get_currency_symbol(),
            price[0],
            self.get_decimal_mark(),
            price[1],
        )

    @view.memoize
    def is_edit_allowed(self):
        """Check if edit is allowed
        """
        return check_permission(FieldEditProfiles, self.context)

    @view.memoize
    def get_editable_columns(self):
        """Return editable fields
        """
        columns = []
        if self.is_edit_allowed():
            columns = ["Hidden"]
        return columns

    def extract(self):
        """Extract the value from the request for the field
        """
        form = self.request.form
        selected = form.get(self.select_checkbox_name, [])

        if not selected:
            return []

        # extract the data from the form for the field
        records = []
        hidden_services = form.get("Hidden", {})
        for uid in selected:
            records.append({
                "uid": uid,
                "hidden": hidden_services.get(uid) == "on",
            })

        return records

    def folderitems(self):
        items = super(AnalysisProfilesWidget, self).folderitems()
        self.categories.sort()
        return items

    def folderitem(self, obj, item, index):
        """Service triggered each time an item is iterated in folderitems.

        The use of this service prevents the extra-loops in child objects.

        :obj: the instance of the class to be foldered
        :item: dict containing the properties of the object to be used by
            the template
        :index: current index of the item
        """
        item = super(AnalysisProfilesWidget, self).folderitem(obj, item, index)

        # ensure we have an object and not a brain
        obj = api.get_object(obj)
        uid = api.get_uid(obj)
        url = api.get_url(obj)
        title = api.get_title(obj)

        # get the category
        if self.show_categories_enabled():
            category = obj.getCategoryTitle()
            if category not in self.categories:
                self.categories.append(category)
            item["category"] = category

        hidden = False
        # get the hidden setting from the records
        if self.records.get(uid):
            record = self.records.get(uid, {}) or {}
            hidden = record.get("hidden", False)
        else:
            # get the default value from the service
            hidden = obj.getHidden()

        item["replace"]["Title"] = get_link(url, value=title)
        item["Price"] = self.format_price(obj.Price)
        item["allow_edit"] = self.get_editable_columns()
        item["selected"] = False
        item["Hidden"] = hidden
        item["replace"]["Hidden"] = _("Yes") if hidden else _("No")
        item["selected"] = uid in self.records
        item["Keyword"] = obj.getKeyword()

        # Add methods
        methods = obj.getMethods()
        if methods:
            links = map(
                lambda m: get_link(
                    m.absolute_url(), value=m.Title(), css_class="link"),
                methods)
            item["replace"]["Methods"] = ", ".join(links)
        else:
            item["methods"] = ""

        # Unit
        unit = obj.getUnit()
        item["Unit"] = unit or ""
        item["replace"]["Unit"] = unit and format_supsub(unit) or ""

        # Icons
        after_icons = ""
        if obj.getAccredited():
            after_icons += get_image(
                "accredited.png", title=_("Accredited"))
        if obj.getAttachmentRequired():
            after_icons += get_image(
                "attach_reqd.png", title=_("Attachment required"))
        if after_icons:
            item["after"]["Title"] = after_icons

        return item


1			# -- coding: utf-8 --
2
3			import collections
4
5			from bika.lims import api
6			from bika.lims import bikaMessageFactory as _
7			from bika.lims.api.security import check_permission
8			from bika.lims.utils import format_supsub
9			from bika.lims.utils import get_image
10			from bika.lims.utils import get_link
11			from plone.memoize import view
12			from senaite.core.catalog import SETUP_CATALOG
13			from senaite.core.permissions import FieldEditProfiles
14			from senaite.core.z3cform.widgets.listing.view import DefaultListingWidget
15			from zope.i18n.locales import locales
16
17
18			class AnalysisProfilesWidget(DefaultListingWidget):
19			"""Listing widget for Analysis Profiles
20			"""
21			def __init__(self, field, request):
22			super(AnalysisProfilesWidget, self).__init__(field, request)
23
24			self.catalog = SETUP_CATALOG
25			self.contentFilter = {
26			"portal_type": "AnalysisService",
27			"sort_on": "sortable_title",
28			"sort_order": "ascending",
29			"is_active": True,
30			}
31
32			# group the current records by UID
33			self.records = {}
34			for record in self.get_value():
35			uid = record.get("uid")
36			self.records[uid] = record
37
38			# listing config
39			self.allow_edit = True
40			self.context_actions = {}
41			self.fetch_transitions_on_select = False
42			self.omit_form = True
43			self.pagesize = 999999
44			self.show_column_toggles = False
45			self.show_search = True
46			self.show_select_all_checkbox = False
47			self.show_select_column = True
48			self.show_table_footer = False
49
50			# Categories
51			if self.show_categories_enabled():
52			self.categories = []
53			self.show_categories = True
54			self.expand_all_categories = False
55
56			self.columns = collections.OrderedDict((
57			("Title", {
58			"title": _(
59			u"listing_analysisprofileswidget_column_title",
60			default=u"Service"
61			),
62			"index": "sortable_title",
63			"sortable": False
64			}),
65			("Keyword", {
66			"title": _(
67			u"listing_analysisprofileswidget_column_keyword",
68			default=u"Keyword"
69			),
70			"sortable": False
71			}),
72			("Methods", {
73			"title": _(
74			u"listing_analysisprofileswidget_column_methods",
75			default=u"Methods"
76			),
77			"sortable": False
78			}),
79			("Unit", {
80			"title": _(
81			u"listing_analysisprofileswidget_column_unit",
82			default=u"Unit"
83			),
84			"sortable": False
85			}),
86			("Price", {
87			"title": _(
88			u"listing_analysisprofileswidget_column_price",
89			default=u"Price"
90			),
91			"sortable": False,
92			}),
93			("Hidden", {
94			"title": _(
95			u"listing_analysisprofileswidget_column_hidden",
96			default=u"Hidden"
97			),
98			"sortable": False,
99			}),
100			))
101
102			cols = self.columns.keys()
103			if not self.show_prices():
104			cols.remove("Price")
105
106			self.review_states = [
107			{
108			"id": "default",
109			"title": _(
110			u"listing_analysisprofileswidget_state_all",
111			default=u"All"
112			),
113			"contentFilter": {},
114			"transitions": [{"id": "disallow-all-possible-transitions"}],
115			"columns": cols,
116			},
117			]
118
119			@view.memoize
120			def show_categories_enabled(self):
121			"""Check in the setup if categories are enabled
122			"""
123			return self.context.bika_setup.getCategoriseAnalysisServices()
124
125			@view.memoize
126			def show_prices(self):
127			"""Checks if prices should be shown or not
128			"""
129			setup = api.get_setup()
130			return setup.getShowPrices()
131
132			@view.memoize
133			def get_currency_symbol(self):
134			"""Get the currency Symbol
135			"""
136			locale = locales.getLocale("en")
137			setup = api.get_setup()
138			currency = setup.getCurrency()
139			return locale.numbers.currencies[currency].symbol
140
141			@view.memoize
142			def get_decimal_mark(self):
143			"""Returns the decimal mark
144			"""
145			setup = api.get_setup()
146			return setup.getDecimalMark()
147
148			@view.memoize
149			def format_price(self, price):
150			"""Formats the price with the set decimal mark and correct currency
151			"""
152			return u"{} {}{}{:02d}".format(
153			self.get_currency_symbol(),
154			price[0],
155			self.get_decimal_mark(),
156			price[1],
157			)
158
159			@view.memoize
160			def is_edit_allowed(self):
161			"""Check if edit is allowed
162			"""
163			return check_permission(FieldEditProfiles, self.context)
164
165			@view.memoize
166			def get_editable_columns(self):
167			"""Return editable fields
168			"""
169			columns = []
170			if self.is_edit_allowed():
171			columns = ["Hidden"]
172			return columns
173
174			def extract(self):
175			"""Extract the value from the request for the field
176			"""
177			form = self.request.form
178			selected = form.get(self.select_checkbox_name, [])
179
180			if not selected:
181			return []
182
183			# extract the data from the form for the field
184			records = []
185			hidden_services = form.get("Hidden", {})
186			for uid in selected:
187			records.append({
188			"uid": uid,
189			"hidden": hidden_services.get(uid) == "on",
190			})
191
192			return records
193
194			def folderitems(self):
195			items = super(AnalysisProfilesWidget, self).folderitems()
196			self.categories.sort()
197			return items
198
199			def folderitem(self, obj, item, index):
200			"""Service triggered each time an item is iterated in folderitems.
201
202			The use of this service prevents the extra-loops in child objects.
203
204			:obj: the instance of the class to be foldered
205			:item: dict containing the properties of the object to be used by
206			the template
207			:index: current index of the item
208			"""
209			item = super(AnalysisProfilesWidget, self).folderitem(obj, item, index)
210
211			# ensure we have an object and not a brain
212			obj = api.get_object(obj)
213			uid = api.get_uid(obj)
214			url = api.get_url(obj)
215			title = api.get_title(obj)
216
217			# get the category
218			if self.show_categories_enabled():
219			category = obj.getCategoryTitle()
220			if category not in self.categories:
221			self.categories.append(category)
222			item["category"] = category
223
224			hidden = False
225			# get the hidden setting from the records
226			if self.records.get(uid):
227			record = self.records.get(uid, {}) or {}
228			hidden = record.get("hidden", False)
229			else:
230			# get the default value from the service
231			hidden = obj.getHidden()
232
233			item["replace"]["Title"] = get_link(url, value=title)
234			item["Price"] = self.format_price(obj.Price)
235			item["allow_edit"] = self.get_editable_columns()
236			item["selected"] = False
237			item["Hidden"] = hidden
238			item["replace"]["Hidden"] = _("Yes") if hidden else _("No")
239			item["selected"] = uid in self.records
240			item["Keyword"] = obj.getKeyword()
241
242			# Add methods
243			methods = obj.getMethods()
244			if methods:
245			links = map(
246			lambda m: get_link(
247			m.absolute_url(), value=m.Title(), css_class="link"),
248			methods)
249			item["replace"]["Methods"] = ", ".join(links)
250			else:
251			item["methods"] = ""
252
253			# Unit
254			unit = obj.getUnit()
255			item["Unit"] = unit or ""
256			item["replace"]["Unit"] = unit and format_supsub(unit) or ""
257
258			# Icons
259			after_icons = ""
260			if obj.getAccredited():
261			after_icons += get_image(
262			"accredited.png", title=_("Accredited"))
263			if obj.getAttachmentRequired():
264			after_icons += get_image(
265			"attach_reqd.png", title=_("Attachment required"))
266			if after_icons:
267			item["after"]["Title"] = after_icons
268
269			return item
270

senaite / senaite.core

Push — 2.x ( 864250...eaa7c8 )

AnalysisProfilesWidget.folderitem() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Complexity

Duplication Side-by-Side

Filter issues like