Project

Profile

Help

Task #5428

Story #5443: [Epic] Remove leading _ from pulpcore model fields

Rename the fields "_relative_path" and "_artifact" on the SingleArtifactContentSerializer to "relative_path" and "artifact" respectively

Added by mdellweg about 1 month ago. Updated about 1 month ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

The content serializers provided by pulpcore-plugin are only convenience layers for the plugin writer. Their _artifact and _relative_path fields do not refer directly to filed on the corresponding content models, but are translated into ContentArtifacts on create. Plugins really exposing the relative_path field take some effort to rename it to relative_path in a way that might even skip some valuable validation.

There has been some consensus that those fields should be named artifact and relative_path.


Related issues

Related to Pulp - Issue #5445: "KeyError: 'relative_path'" in SingleArtifactContentSerializer MODIFIED Actions

History

#1 Updated by mdellweg about 1 month ago

  • Assignee set to mdellweg

#2 Updated by mdellweg about 1 month ago

  • Status changed from NEW to MODIFIED
  • % Done changed from 0 to 100

#3 Updated by daviddavis about 1 month ago

  • Status changed from MODIFIED to POST

#4 Updated by daviddavis about 1 month ago

  • Status changed from POST to MODIFIED
  • Sprint/Milestone set to 71

#5 Updated by dkliban@redhat.com about 1 month ago

  • Parent task set to #5443

#6 Updated by daviddavis about 1 month ago

  • Related to Issue #5445: "KeyError: 'relative_path'" in SingleArtifactContentSerializer added

#7 Updated by bmbouter about 1 month ago

  • Sprint/Milestone changed from 71 to 3.0

Please register to edit this issue

Also available in: Atom PDF