https://dev.e-taxonomy.eu/redmine/https://dev.e-taxonomy.eu/redmine/redmine/favicon.ico?14691914852021-01-20T17:03:52ZEDIT Project ManagementEDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=573012021-01-20T17:03:52ZAndreas Müller
<ul><li><strong>Copied from</strong> <i><a class="issue tracker-4 status-5 priority-10 priority-lowest closed child" href="/redmine/issues/3737">bug #3737</a>: Protected caches for taxa and names behave inconsistent</i> added</li></ul> EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=573022021-01-20T17:04:11ZAndreas Müller
<ul><li><strong>Subject</strong> changed from <i>Fields set to irrelevant should be set by state only</i> to <i>Fields set to irrelevant for cache should be set by state only</i></li><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=573062021-01-20T17:06:29ZAndreas Müller
<ul><li><strong>Related to</strong> <i><a class="issue tracker-4 status-5 priority-13 priority-high13 closed child" href="/redmine/issues/4944">bug #4944</a>: "Protected" indication for references needs to be improved</i> added</li></ul> EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=575272021-01-23T01:19:33ZAndreas Müller
<ul></ul><p>This is mostly fixed. Some advanced features stay:</p>
<ul>
<li>trigger updateCacheRelevance() with ToggledText state change listener (probably this is the SelectionListener)</li>
<li>improve handling of team members in Team element. (by setting a breakpoint somewhere at TeamMemberElement creation it becomes clearer how to insert registration</li>
<li>improve colors</li>
<li>handle also for other DetailElements like Collection detail element</li>
<li>improve use of ICacheRelevantFormElement in class hierarchies (probably the use can still be reduced)</li>
<li>implement RefType dependency (mostly of abbrevTitleCache) in ReferenceDetail view (also depending on cache strategy) </li>
<li>Very advanced: use cache strategy to define which fields are relevant</li>
</ul>
EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=575282021-01-23T01:21:48ZAndreas Müller
<ul><li><strong>Copied to</strong> <i><a class="issue tracker-5 status-1 priority-11 priority-default" href="/redmine/issues/9417">feature request #9417</a>: Advanced issues for cache relevance handling</i> added</li></ul> EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=575302021-01-23T01:23:12ZAndreas Müller
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li><li><strong>Assignee</strong> changed from <i>Andreas Müller</i> to <i>Katja Luther</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>50</i></li></ul><p>This should be fixed. I moved advanced issues to new ticket <a class="issue tracker-5 status-1 priority-11 priority-default" title="feature request: Advanced issues for cache relevance handling (New)" href="https://dev.e-taxonomy.eu/redmine/issues/9417">#9417</a>. Please review.</p>
<p>This is also related to <a class="issue tracker-4 status-5 priority-13 priority-high13 closed child" title="bug: "Protected" indication for references needs to be improved (Closed)" href="https://dev.e-taxonomy.eu/redmine/issues/4944">#4944</a>. Please include this in review.</p>
EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=575332021-01-23T01:26:15ZAndreas Müller
<ul><li><strong>Related to</strong> <i><a class="issue tracker-4 status-5 priority-16 priority-high10 closed child" href="/redmine/issues/4915">bug #4915</a>: Abbrev title for person should never be orange</i> added</li></ul> EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=575352021-01-23T01:39:51ZAndreas Müller
<ul></ul><p>The general idea of the refactoring is:</p>
<ul>
<li>in a first step all elements which are relevant for a cache get do register the cache the depend on.</li>
<li>when the caches change the updateCacheRelevance() method is triggered for all such elements (this is currently still done with a cascading call, but better we should use a state change listener on the toggle/cache field (<a class="issue tracker-5 status-1 priority-11 priority-default" title="feature request: Advanced issues for cache relevance handling (New)" href="https://dev.e-taxonomy.eu/redmine/issues/9417">#9417</a>) in future to make it all still more state dependend.</li>
<li>each cache field has a registered color (CacheRelevance), so if 2 different cache fields exist, it can be indicated by color for which cache the field is irrelevant (or for both with 3rd color)</li>
<li>fields only indirectly irrelevant (e.g. uninomial for an unprotected namecache but a protected titleCache) use a light color</li>
<li>as the relevant elements and sections do not all inherit from the same base classes a helper class CacheRelevanceHelper exists which handled typical method calls (decorator pattern)</li>
</ul>
EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=576252021-01-26T13:25:25ZKatja Luther
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li><li><strong>Assignee</strong> changed from <i>Katja Luther</i> to <i>Andreas Müller</i></li></ul><p>This seems to work correctly (also for references), I only added the colouring of the date field for references to <a class="issue tracker-5 status-1 priority-11 priority-default" title="feature request: Advanced issues for cache relevance handling (New)" href="https://dev.e-taxonomy.eu/redmine/issues/9417">#9417</a>.</p>
EDIT - bug #9405: Fields set to irrelevant for cache should be set by state onlyhttps://dev.e-taxonomy.eu/redmine/issues/9405?journal_id=576262021-01-26T13:25:36ZKatja Luther
<ul><li><strong>% Done</strong> changed from <i>50</i> to <i>100</i></li></ul>