Expression doesnt work in ng-maxlength

[hyusetiawan]

here is the plink: http://plnkr.co/edit/FgmK1b?p=preview&s=tempoooo

checks the maxlength that is 5 using an expression, and it's not validated however, using {{expression}} it shows the value

[hyusetiawan]

the same thing happens to max validation, makes me wonder, does the validation directive accept expression? if not, how do we properly pass parameters to them?

[hkdobrev]

👍 I think I have found the problem here: https://github.com/angular/angular.js/blob/master/src/ng/directive/input.js#L494

The ngMaxlength, ngMinlength and perhaps other pseudo-directives used by the input directive do not evaluate the value of the attribute in the current scope, but just cast it to a integer.

Any input from the AngularJS core developers? @IgorMinar @btford

I think a int(scope.$eval(attr.ngMaxlength)) would do the job. What do you think?

[timneedham]

I would love to see this get fixed as well, the suggestion by @hkdobrev works great for me...

I'm new to these parts (loving AngularJS!), but it seems things may have stalled on this tiny issue? So as above, any input from the AngularJS core developers would be gratefully received! @IgorMinar @btford

[bowsersenior]

Just ran into this same problem and wasted a lot of time. It would be awesome if this could get fixed soon. Thanks!

[bowsersenior]

Oh, just noticed #1588 was closed a few months back due to failing tests. Anybody care to have a second look at that pull request to get the tests passing?

[blowsie]

+1 for fixing this! Is it planned to be fixed for version 1.2? or anytime soon?

[the-spyke]

Any progress on this?

[hkdobrev]

Hey, what's up with this one? 😄

https://github.com/blog/1721-the-ghost-of-issues-past

[anpsince83]

Another +1 for this. Seems like an essential feature for dynamic forms.

[blowsie]

Also should this really be tagged as a feature and not a bug?

[hkdobrev]

@blowsie It is not documented as if it's working, so it's not a bug.

[elranu]

👍 Another +1 for this. Seems like an essential feature for dynamic forms.

[MagRelo]

holla! +1