Skip to content

Replace generic updateOrRemove service with ProductFeatureDataResource-specific service#1227

Merged
dixitdeepak merged 1 commit into
apache:trunkfrom
dixitdeepak:replaced-ambiguous-method-updateOrRemove
May 19, 2026
Merged

Replace generic updateOrRemove service with ProductFeatureDataResource-specific service#1227
dixitdeepak merged 1 commit into
apache:trunkfrom
dixitdeepak:replaced-ambiguous-method-updateOrRemove

Conversation

@dixitdeepak
Copy link
Copy Markdown
Contributor

Replaced the ambiguous updateOrRemove service with a dedicated createOrRemoveProductFeatureDataResource service for managing CMS product feature data resource links.

  • Added explicit productFeatureId and dataResourceId service parameters
  • Removed generic entityName, pkFieldCount, fieldName, and fieldValue handling
  • Simplified ProductFeatureDataResource create/remove logic
  • Updated CMSContentEdit.ftl to submit defined service parameters
  • Updated updateFeatures controller event to invoke the new service

…e-specific service

Replaced the ambiguous updateOrRemove service with a dedicated createOrRemoveProductFeatureDataResource service for managing CMS product feature data resource links.

- Added explicit productFeatureId and dataResourceId service parameters
- Removed generic entityName, pkFieldCount, fieldName, and fieldValue handling
- Simplified ProductFeatureDataResource create/remove logic
- Updated CMSContentEdit.ftl to submit defined service parameters
- Updated updateFeatures controller event to invoke the new service
Copy link
Copy Markdown
Contributor

@jacopoc jacopoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me

@dixitdeepak dixitdeepak merged commit 60e4f25 into apache:trunk May 19, 2026
5 checks passed
dixitdeepak added a commit that referenced this pull request May 19, 2026
…e-specific service (#1227)

Replaced the ambiguous updateOrRemove service with a dedicated
createOrRemoveProductFeatureDataResource service for managing CMS
product feature data resource links.

- Added explicit productFeatureId and dataResourceId service parameters
- Removed generic entityName, pkFieldCount, fieldName, and fieldValue
handling
- Simplified ProductFeatureDataResource create/remove logic
- Updated CMSContentEdit.ftl to submit defined service parameters
- Updated updateFeatures controller event to invoke the new service
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants