Remove $bucketId paramater from S3fsMetadataServiceInterface::writeCache

Created on 24 February 2023, almost 2 years ago

Updated 26 February 2023, almost 2 years ago

Problem/Motivation

We currently have all the information we need inside of $metadata['bucket'] that requiring $bucketId is a duplication of effort that may also cause a potential data mismatch.

Steps to reproduce

Review S3fsMetadataService::writeCache()

Proposed resolution

Remove $bucketId and replace with $metadata['bucket']

Remaining tasks

Patch

User interface changes

None

API changes

Remove required paramater $bucketId.

Data model changes

None

📌 Task

Status

Fixed

Version

4.0

Component

Code

Created by

🇺🇸United States cmlara

Live updates comments and jobs are added and updated live.

beta blocker

Comments & Activities

Issue created by @cmlara
Comment almost 2 years ago →
System Message

cmlara → committed 687ebbdb on 4.0.x
Issue #3344176: Remove $bucketId paramater from...
Status changed to Fixed almost 2 years ago10:59pm 26 February 2023
Comment almost 2 years ago →
🇺🇸United States cmlara
Comment almost 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024