Skip to content
This repository has been archived by the owner on Nov 20, 2018. It is now read-only.

No easy way to get file names in IFormFileCollection / IFormFile #499

Closed
Yves57 opened this issue Dec 4, 2015 · 4 comments
Closed

No easy way to get file names in IFormFileCollection / IFormFile #499

Yves57 opened this issue Dec 4, 2015 · 4 comments
Assignees
@davidfowl
Labels
3 - Done enhancement
Milestone
1.0.0-rc2

Comments

@Yves57
Copy link
Contributor

Yves57 commented Dec 4, 2015

When using IFormCollection, it is easy to get the form keys.
But when working with the IFormFileCollection interface, there is not way to get all form names and file names (somehing like IFormCollection.Keys / IFormCollection.FileNames) or at least two properties IFormFile.FileName and IFormFile.Name.
I have found a workaround (using ContentDispositionHeaderValue like in the FormFileCollection class, but it is not very fun to use.
@davidfowl
Copy link
Member

/cc @Tratcher @muratg

@davidfowl
Copy link
Member

We need to expose the Name on IFormFile.

@khellang khellang mentioned this issue Dec 4, 2015
Merged
@khellang
Copy link
Contributor

khellang commented Dec 4, 2015

@davidfowl Submitted #500

khellang added a commit to khellang/HttpAbstractions that referenced this issue Dec 4, 2015
@khellang
Added Name and FileName to IFormFile
c2e7618 
This commits also gets rid of the name
closure in FormFileCollection by interating
over the files in the collection instead
of using Find and FindAll.

Closes aspnet#352 and aspnet#499
@davidfowl davidfowl self-assigned this Dec 4, 2015
@davidfowl davidfowl added this to the 1.0.0-rc2 milestone Dec 4, 2015
@Yves57
Copy link
Contributor Author

Yves57 commented Dec 5, 2015

Only few hours to fix the issue. Thank you!

@rynowak rynowak mentioned this issue Jan 12, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@davidfowl davidfowl

Labels
3 - Done enhancement
Projects
None yet
Milestone
1.0.0-rc2
Development

No branches or pull requests
3 participants
@davidfowl @khellang @Yves57