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

Revive ByteArrayModelBinder #728

Closed
kevinchalet opened this issue Jul 1, 2014 · 3 comments
Closed

Revive ByteArrayModelBinder #728

kevinchalet opened this issue Jul 1, 2014 · 3 comments
Assignees
@sornaks
Labels
3 - Done feature
Milestone
1.0.0-alpha4

Comments

@kevinchalet
Copy link
Contributor

MVC vPrevious (:trollface:) was able to bind byte[] properties thanks to its special ByteArrayModelBinder.

In MVC 6, ByteArrayModelBinder has not been ported yet and byte[] properties cannot be bound correctly: the bytes array is instantiated but its length is incorrect (always 1) and doesn't reflect the base64-encoded string created by the HTML helpers and retrieved by the value providers.

Please consider porting ByteArrayModelBinder.
@yishaigalatzer yishaigalatzer added this to the 6.0.0-alpha4 milestone Jul 23, 2014
@sornaks
Copy link

sornaks commented Aug 13, 2014

Checked in: 9c4d780

@sornaks sornaks closed this as completed Aug 13, 2014
@davidfowl
Copy link
Member

This doesn't have a bug/feature tag @yishaigalatzer @danroth27

@yishaigalatzer
Copy link
Contributor

Thanks @pranavkm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sornaks sornaks

Labels
3 - Done feature
Projects
None yet
Milestone
1.0.0-alpha4
Development

No branches or pull requests
6 participants
@davidfowl @pranavkm @yishaigalatzer @danroth27 @sornaks @kevinchalet