Project

General

Profile

feature request #5972

Prevent user from unwanted author and reference updates

Added by Andreas Müller almost 3 years ago. Updated 9 months ago.

Status:
Feedback
Priority:
Priority13
Assignee:
Category:
taxeditor
Target version:
Start date:
07/08/2016
Due date:
% Done:

80%

Estimated time:
4.00 h
Severity:
critical

Description

From time to time it happens that users press the "edit" button for editing authors or references.

This is usually unwanted and very risky as the record might be used at many places. We expect that changes should usually take place only in the bulk editor. The only usecase where it is wanted to edit immediately is directly after creating a new record to correct it. E.g. after incorrect parsing it might be wanted.

There are multiple solutions:

  • Make availability of editing button a preference with default to "not available".

  • Check if record is persisted (id > 0), if not, make button available as this is without risk - see entity.isPersisted()

  • Inform the user via dialog box or with red, bold warning that he/she might edit a record that is used elsewhere

  • Check if the record is used elsewhere (via referencing objects). Only allow editing if not used elsewhere (this might be a performance issue the request may take time, we may need a new method that only checks IF the record is used elsewhere, not how and not with initializing any records)

  • same as before but method is only called when record is opened for editing

  • we might also handle this via rights&roles but probably this is not adequate. Generally we allow those users to update agent/reference records but only in bulk editor. We would need TaxEditor specific rights to do so.


Related issues

Related to Edit - bug #6176: Creating a new Reference throws TOE (TransientObjectException) Closed 11/02/2016 11/09/2016
Related to Edit - bug #6178: TOE Exception when creating a new Reference Duplicate 11/02/2016
Related to AlgenRegistrierung - bug #6185: prevent from erroneous author or reference changes Resolved 11/07/2016
Related to Edit - feature request #7706: Warning for reused Media is missing Closed 08/30/2018

Associated revisions

Revision 683f6ce0 (diff)
Added by Katja Luther over 2 years ago

ref #5972: add method to count referencing objects

Revision a8b0ad5f (diff)
Added by Katja Luther over 2 years ago

ref #5972: add service method to get count of referencing objects

Revision 84269784 (diff)
Added by Katja Luther over 2 years ago

ref #5972: the wizard warns the user if the reference is used ssomewhere else

History

#1 Updated by Andreas Müller almost 3 years ago

  • Assignee changed from Andreas Müller to k.luther -

#2 Updated by Andreas Müller over 2 years ago

  • Description updated (diff)
  • Priority changed from Highest to Priority13
  • Target version changed from Unassigned CDM tickets to Release 4.4
  • Estimated time set to 4.00 h

Decision: for id > 0 implement check for further references and inform the user in red about these references if they exist

#3 Updated by Katja Luther over 2 years ago

  • Status changed from New to In Progress
  • Assignee changed from Katja Luther to Andreas Müller

@AM Please have a look whether the message is ok or if it should be bigger or other text...

#4 Updated by Katja Luther over 2 years ago

  • % Done changed from 0 to 80

#5 Updated by Katja Luther over 2 years ago

  • Related to bug #6175: Exception when deleting transient object added

#6 Updated by Katja Luther over 2 years ago

  • Related to bug #6176: Creating a new Reference throws TOE (TransientObjectException) added

#7 Updated by Katja Luther over 2 years ago

  • Related to deleted (bug #6175: Exception when deleting transient object)

#8 Updated by Andreas Müller over 2 years ago

  • Status changed from In Progress to Resolved

#9 Updated by Andreas Müller over 2 years ago

  • Related to bug #6178: TOE Exception when creating a new Reference added

#10 Updated by Andreas Kohlbecker over 2 years ago

  • Related to bug #6185: prevent from erroneous author or reference changes added

#11 Updated by Andreas Müller over 2 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther

I would change the label to "This reference is referenced by xxx records. The change will take for all these records."

Make it bold.

Background should be gray, as it is not an editable part (like for other labels, too).

Have more spacing around the label.

#12 Updated by Andreas Müller over 2 years ago

It looks like it wokrs only for references until now. But we need it for all objects (if this is difficult, at least for authors).

#13 Updated by Andreas Müller over 2 years ago

  • Private changed from Yes to No

#14 Updated by Andreas Müller 12 months ago

I think this is fixed or mostly fixed. Can you please check and give me back for review if you think it is fixed?

#15 Updated by Andreas Müller 9 months ago

#16 Updated by Andreas Kohlbecker 9 months ago

  • Description updated (diff)

we might also handle this via rights&roles but probably this is not adequate. Generally we allow those users to update agent/reference records but only in bulk editor. We would need TaxEditor specific rights to do so.

I think a combination of appropriate permissions and blocking the modification of entities which are used elsewhere would be a suitable solution. see #4305 for an in depth discussion

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)