[MNG-7634] Revert MNG-5982 and MNG-7417

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.8.7, 3.9.0, 4.0.0-alpha-4, 4.0.0
• Assignee: Michael Osipov
• Reporter: Michael Osipov
• Created: 17/Dec/22 19:30
• Updated: 27/Dec/22 08:13
• Resolved: 20/Dec/22 08:25

Description

As layed out in MNG-7563, both issues though solve one problem, but cause a regression where direct user input manipulates dependency POMs in a way which is not clear/documented as such. As long as we cannot properly understand and document it, we must revert those changes.

Issue Links

• causes: MNG-7635 Investigate how to safely propagate user properties to dependencies (see MNG-5982 and MNG-7417)
• fixes: MNG-7563 REGRESSION: User properties now override model properties in dependencies
• supercedes: MNG-7417 Several classes do not set properties properly for building requests
• supercedes: MNG-5982 The POM for ... is invalid, transitive dependencies ... while property was overriden
• links to: GitHub Pull Request #916
• links to: GitHub Pull Request #917