Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix undefined index in stat metadata #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix undefined index in stat metadata #15

wants to merge 2 commits into from

Conversation

kor3k
Copy link

@kor3k kor3k commented Aug 26, 2019

fix undefined index in stat metadata #12

@twistor
Copy link
Owner

twistor commented Nov 4, 2019

What key is not defined, and what adapter are you using?

@0b10011
Copy link

0b10011 commented Nov 4, 2019

@twistor see #12:

PHP Notice: Undefined offset: 0 in <SOME_PATH>/vendor/twistor/flysystem-stream-wrapper/src/Flysystem/Plugin/Stat.php on line 157

@kor3k
Copy link
Author

kor3k commented Nov 4, 2019

@twistor yes, it's described here #12 (comment)

rulatir added a commit to rulatir/flysystem-stream-wrapper that referenced this pull request Nov 15, 2019
Fix/metadata undefined index - merge twistor#15 recreated against my fork
@lsv
Copy link

lsv commented Jun 4, 2020

@twistor

It would be awesome if this could get merged. It actually fixes if you are using memory adapter

@Renrhaf
Copy link

Renrhaf commented Jun 8, 2020

Hello! I would need this patch too for my API platform backend.

@hexus
Copy link

hexus commented Jul 28, 2020

I too need this fixed, please merge and release 🙏

@DieterHolvoet
Copy link

DieterHolvoet commented Oct 20, 2023

I just encountered this error as well, having the current PR applied as patch:

Warning: Undefined array key "type" in Twistor\Flysystem\Plugin\Stat->mergeMeta() (line 157 of vendor/twistor/flysystem-stream-wrapper/src/Flysystem/Plugin/Stat.php)

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.

7 participants