You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bloodhound.apache.org by Apache Bloodhound <de...@bloodhound.apache.org> on 2013/07/17 10:01:19 UTC
[Apache Bloodhound] #595: Refactor
multiproduct.product.ProductEnvironment to remove private caching
variables
#595: Refactor multiproduct.product.ProductEnvironment to remove private caching
variables
------------------------------+-----------------------
Reporter: rjollos | Owner: rjollos
Type: enhancement | Status: new
Priority: minor | Milestone: Release 7
Component: multiproduct | Version:
Keywords: environment lazy |
------------------------------+-----------------------
Discussed in #592, since `href` and `abs_href` are decorated with `@lazy`,
the methods can be refactored to eliminate the private `_href` and
`_abs_href` attributes. Currently we effectively have two methods in place
of caching the computed value. See also t:#11245.
--
Ticket URL: <https://issues.apache.org/bloodhound/ticket/595>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker
Re: [Apache Bloodhound] #595: Refactor
multiproduct.env.ProductEnvironment to remove private caching
variables
Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#595: Refactor multiproduct.env.ProductEnvironment to remove private caching
variables
---------------------------+------------------------------
Reporter: rjollos | Owner: rjollos
Type: enhancement | Status: closed
Priority: minor | Milestone: Release 7
Component: multiproduct | Version:
Resolution: fixed | Keywords: environment lazy
---------------------------+------------------------------
Changes (by rjollos):
* status: accepted => closed
* resolution: => fixed
Comment:
Changes are similar to [t 11863:11867].
----
(In [1511622])
`multiproduct.env.ProductEnvironment`: Removed private `_href` variable,
made unnecessary by use of `lazy` decorator. Refs #595.
----
(In [1511625])
`multiproduct.env.ProductEnvironment`: Removed private `_abs_href`
attribute, made unnecessary by use of `lazy` decorator. Refs #595.
-----
(In [1511628])
`multiproduct.env.ProductEnvironment`: Add unit test for `path` attribute
laziness. Refs #595.
-----
(In [1511640])
`multiproduct.env.ProductEnvironment`: Refactoring - Access `path` value
in `SplitResult` object by attribute (available since Python 2.5) rather
than index, to improve code clarity.
Added tests for `href` attribute. Refs #595.
--
Ticket URL: <https://issues.apache.org/bloodhound/ticket/595#comment:2>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker
Re: [Apache Bloodhound] #595: Refactor
multiproduct.env.ProductEnvironment to remove private caching
variables (was: Refactor multiproduct.product.ProductEnvironment
to remove private caching variables)
Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#595: Refactor multiproduct.env.ProductEnvironment to remove private caching
variables
---------------------------+------------------------------
Reporter: rjollos | Owner: rjollos
Type: enhancement | Status: accepted
Priority: minor | Milestone: Release 7
Component: multiproduct | Version:
Resolution: | Keywords: environment lazy
---------------------------+------------------------------
Changes (by rjollos):
* status: new => accepted
--
Ticket URL: <https://issues.apache.org/bloodhound/ticket/595#comment:1>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker