Project

General

Profile

Actions

Bug #7526

closed

Bridge should reject child reports that overwrite values of parent report attributes

Added by Evgeny Novikov over 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
High
Category:
Bridge
Target version:
-
Start date:
08/23/2016
Due date:
02/16/2017
% Done:

100%

Estimated time:
Detected in build:
svn
Platform:
Published in build:

Description

Now this isn't the case - Bridge just combine these values together.

This issue was revealed when trying to recalculate statistics caches for jobs where MAV was involved (#7443).


Related issues 1 (0 open1 closed)

Related to Klever - Bug #8020: Bridge does not corrupt job when Core provides incorrect report attributesClosedVladimir Gratinskiy03/09/201706/13/2017

Actions
Actions #1

Updated by Vladimir Gratinskiy about 7 years ago

  • Due date set to 02/16/2017
  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Fixed in "fix_7526". Uploading report would fail if attributes are redefining (but attributes would be saved so user can see what attribute is redefined).

Actions #2

Updated by Evgeny Novikov about 7 years ago

  • Status changed from Resolved to Open

I still observe this issue perhaps due to it was incorrectly formulated. You need to check not only parent report attributes, but all attributes up to the root report. Redefinition any of attribute should result to job corruption.

Actions #3

Updated by Vladimir Gratinskiy about 7 years ago

  • Status changed from Open to Resolved

Fixed for all parents.

Actions #4

Updated by Evgeny Novikov about 7 years ago

  • Status changed from Resolved to Open

Vladimir Gratinskiy wrote:

Fixed for all parents.

This fix resulted in original behavior. Bridge concatenates attribute values with the same name when these attributes come from a component and a parent of its parent.

Actions #5

Updated by Vladimir Gratinskiy about 7 years ago

  • Status changed from Open to Resolved

Fixed for leaves also.

Actions #6

Updated by Evgeny Novikov about 7 years ago

  • Status changed from Resolved to Closed

I merged the branch to master in 48e535e.

Actions

Also available in: Atom PDF