Modify ↓

Opened 3 years ago

Closed 3 years ago

#21124 closed enhancement (fixed)

[PATCH] Better fix for #20888 and #21117.

Reported by: marcello@… Owned by: Don-vip
Priority: normal Milestone: 21.07
Component: Core Version:
Keywords: Cc:

Description

This is a simpler and more correct fix for #20888 and #21117.

The root of the problem is that the test if (source != this) { is insufficient when more than one calculated field is in the dialog. Two calculated fields will just fire at each other and thus bypass the test.

The correct fix is: calculated fields should never fire.

Attachments (1)

21124.patch​ (2.2 KB ) - added by marcello@… 3 years ago.

Download all attachments as: .zip

Change History (3)

by marcello@…, 3 years ago

Attachment: 21124.patch​ added

comment:1 by Don-vip, 3 years ago

Milestone: → 21.07
Owner: changed from team to Don-vip
Status: new → assigned

comment:2 by Don-vip, 3 years ago

Resolution: → fixed
Status: assigned → closed

In 18040/josm:

fix #21124 - see #21117 - better fix (patch by marcello)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Don-vip.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.