Project

General

Profile

feature request #7235

Make available area vocabularies a database preference for TaxEditor distribution editor

Added by Andreas Müller over 1 year ago. Updated 7 months ago.

Status:
Closed
Priority:
New
Category:
taxeditor
Target version:
Start date:
02/01/2018
Due date:
% Done:

100%

Severity:
normal
Tags:

Description

See also similar implementation for Vaadin distribution editor (part of #6903).

Vaadin example exists for cuba.

Alternatively we may define a list of single terms, but probably we should leave it as it is and later used defined term lists/sets once they exist.


Related issues

Related to Edit - bug #6159: improve distribution editor in TaxEditor Closed 10/24/2016
Related to Edit - feature request #6903: Requirements for checklisteditor of the German states In Progress 08/10/2017 10/31/2017
Related to Edit - bug #7236: Make available distribution status a database preference for TaxEditor distribution editor Feedback 02/01/2018
Related to Edit - feature request #7793: Master ticket for E+M editor issues Feedback 09/27/2018
Related to Edit - feature request #3989: [DISCUSS] Allow defining the vocabularies to be offered in the select dialogs Resolved 01/24/2014
Related to Edit - feature request #7063: Improve the area selection wizard for distribution editor Closed 11/07/2017
Copied to Edit - bug #7856: Allow local override for distribution vocabularies Closed 10/23/2018

Associated revisions

Revision d4db4070 (diff)
Added by Patrick Plitzner 10 months ago

fix #7235 Fix "allowOverride" flag for common name areas DB prefs

History

#1 Updated by Andreas Müller over 1 year ago

  • Description updated (diff)

#2 Updated by Andreas Müller over 1 year ago

  • Related to bug #6159: improve distribution editor in TaxEditor added

#3 Updated by Andreas Müller over 1 year ago

#4 Updated by Andreas Müller over 1 year ago

  • Related to bug #7236: Make available distribution status a database preference for TaxEditor distribution editor added

#5 Updated by Andreas Müller over 1 year ago

  • Target version changed from Release 4.14 to Release 5.0

#6 Updated by Katja Luther over 1 year ago

  • Status changed from New to In Progress

#7 Updated by Katja Luther over 1 year ago

  • Target version changed from Release 5.0 to Release 5.1

#8 Updated by Katja Luther about 1 year ago

  • Target version changed from Release 5.1 to Release 5.2
  • % Done changed from 0 to 30

#9 Updated by Andreas Müller about 1 year ago

  • Target version changed from Release 5.2 to Release 5.3

#10 Updated by Katja Luther 12 months ago

  • Target version changed from Release 5.3 to Release 5.4

#11 Updated by Andreas Müller 10 months ago

  • Tags set to euro+med
  • Status changed from In Progress to Resolved
  • Assignee changed from Katja Luther to Andreas Müller
  • % Done changed from 30 to 60

I guess this is implemented with #7793 so I put to resolved for review

#12 Updated by Andreas Müller 10 months ago

#13 Updated by Andreas Müller 10 months ago

  • Related to feature request #3989: [DISCUSS] Allow defining the vocabularies to be offered in the select dialogs added

#14 Updated by Andreas Müller 10 months ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Patrick Plitzner
  • % Done changed from 60 to 100

This works fine now. There are open issues with the setting of the local areas to be used. But this is another ticket.

Very good is, that local area selections are removed if the according vocabulary is removed from the DB preferences.

Only "local override" does not yet exist and therefore should the checkbox in DB prefs should be disabled.

(Note: local preferences currently only allow selection of single areas within the given vocabulary/ies) .

Patrick, can you please disable the checkbox and always set it to false if, and only if, a value for "distribution.area.voc" exists? Only if no vocabulary is selected it should be set to true, as true is always the default (once the default values are implemented correctly)

#15 Updated by Andreas Müller 10 months ago

  • % Done changed from 100 to 90

#16 Updated by Andreas Müller 10 months ago

#17 Updated by Andreas Müller 10 months ago

Andreas Müller wrote:

Only "local override" does not yet exist and therefore should the checkbox in DB prefs should be disabled.

(Note: local preferences currently only allow selection of single areas within the given vocabulary/ies) .

Patrick, can you please disable the checkbox and always set it to false if, and only if, a value for "distribution.area.voc" exists? Only if no vocabulary is selected it should be set to true, as true is always the default (once the default values are implemented correctly)

Sorry, this was partly not correct. Local preferences do allow setting vocabularies, too. However it does not work as expected: #7856.

Relevant for this ticket is only that currently the "allowOverride" is not saved to the DB if it is set to "true". This still needs to be fixed. Disabling the checkbox is not necessary.

#18 Updated by Andreas Müller 10 months ago

  • Copied to bug #7856: Allow local override for distribution vocabularies added

#19 Updated by Patrick Plitzner 10 months ago

  • Status changed from Feedback to Resolved
  • % Done changed from 90 to 50

#20 Updated by Patrick Plitzner 10 months ago

  • Assignee changed from Patrick Plitzner to Andreas Müller
  • % Done changed from 50 to 90

#21 Updated by Andreas Müller 7 months ago

  • Status changed from Resolved to Closed
  • Assignee changed from Andreas Müller to Patrick Plitzner
  • % Done changed from 90 to 100

I think we can close this ticket. All open (and maybe new issues) on handling the Pref in the UI will be handled in #7856 or #7849

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)