Project

Profile

Help

Issue #3410

closed

Rename content field "path" to "relative_path"

Added by kersom over 6 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Low
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
1. Low
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Documentation
Sprint:
Sprint 33
Quarter:

Description

When reading content from a repository, the response is:

 {'next': None,
 'previous': None,
 'results': [{'_href': 'http://f27-p3:8000/api/v3/content/file/44f4dfa2-1b80-459c-960b-079c4dd6e48c/',
              'artifacts': {'2.iso': 'http://f27-p3:8000/api/v3/artifacts/cbce4558-59b9-4674-83d2-7f6dd2875b6e/'},
              'digest': '3a1b7f18cb8e51f381a9ceb32447fcdaa070d41bbf2d9bd12f5ea1731dfe0b0b',
              'notes': {},
              'path': '2.iso',
              'type': 'file'}]}

</code>

"path" should be renamed to "relative_path" to better denote its purpose.


Related issues

Related to File Support - Issue #3425: Path on content unit doesn't seem to do anythingCLOSED - DUPLICATEActions
Actions #1

Updated by dalley over 6 years ago

I believe "path" is actually short for "relative_path" and that with more complex plugins with directories you would indeed have some which are not just the standalone filename. This isn't apparent with the file plugin because it doesn't do anything like that.

I agree that maybe this should be made explicit.

Actions #2

Updated by amacdona@redhat.com over 6 years ago

+1 rename to relative_path

Actions #3

Updated by bmbouter over 6 years ago

+1 to relative_path

Actions #4

Updated by bmbouter over 6 years ago

  • Subject changed from Rename variable path to name - reading content from a repository to Rename variable path to relative_name when reading content from a repository
Actions #5

Updated by dalley over 6 years ago

  • Subject changed from Rename variable path to relative_name when reading content from a repository to Rename variable path to relative_path when reading content from a repository
  • Description updated (diff)
  • Triaged changed from No to Yes
Actions #6

Updated by amacdona@redhat.com over 6 years ago

  • Project changed from Pulp to File Support
  • Subject changed from Rename variable path to relative_path when reading content from a repository to Rename content field "path" to "relative_path"
  • Sprint/Milestone set to 56
Actions #7

Updated by daviddavis over 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis
Actions #8

Updated by daviddavis over 6 years ago

  • Related to Issue #3425: Path on content unit doesn't seem to do anything added
Actions #9

Updated by daviddavis over 6 years ago

  • Status changed from ASSIGNED to POST

Added by daviddavis over 6 years ago

Revision 30afa7cc | View on GitHub

Limit FileContent api fields to path and artifact

This solves a number of issues such as differences between digest on FileContent vs Artifact.sha256, and FileContent.relative_path vs ContentArtifact.relative_path.

This also renames FileContent.path to FileContent.relative_path

fixes #3428, #3410 https://pulp.plan.io/issues/3428 https://pulp.plan.io/issues/3410

Actions #10

Updated by daviddavis over 6 years ago

  • Status changed from POST to MODIFIED
Actions #11

Updated by bmbouter over 6 years ago

  • Sprint set to Sprint 33
Actions #12

Updated by bmbouter over 6 years ago

  • Sprint/Milestone deleted (56)
Actions #13

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #14

Updated by bmbouter almost 5 years ago

  • Sprint/Milestone set to 0.1.0
Actions #15

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF