79

This seems like it should be fairly easy, but I'm not finding the answer. I have a form where I need to validate that a selection has been made from a radio group. I tried using the 'required' attribute on the radio buttons, but when the form is validated it complains unless all the radio buttons are selected (which is impossible by design).

What is the proper way to validate a radio group selection in AngularJS?

<form name="myForm" ng-submit="submitForm()" ng-controller="ExampleController">
  <input type="radio" ng-model="color" value="red" required>  Red <br/>
  <input type="radio" ng-model="color" value="green" required> Green <br/>
  <input type="radio" ng-model="color" value="blue" required> Blue <br/>
  <tt>color = {{color | json}}</tt><br/>
  <button type="submit">Submit</button>
</form>

Clicking the submit button in the Plnkr shows the behavior.

http://plnkr.co/edit/3qcIbMvJk19OvokcHe2N?p=preview

Improve this question

4 Answers 4

Reset to default
128

Try using ng-required="!color". This makes it so that the field is only required when the value is not set. If a value is set, then required is removed and it will pass validation.

<input type="radio" ng-model="color" value="red" ng-required="!color">  Red <br/>
<input type="radio" ng-model="color" value="green" ng-required="!color"> Green <br/>
<input type="radio" ng-model="color" value="blue" ng-required="!color"> Blue <br/>

Here is an updated plunker that demonstrates that the form now validates correctly: http://plnkr.co/edit/EdItU2IIkO1KIsC052Xx?p=preview

Update

e-cloud's answer is simple and doesn't require an additional directive. I suggest everyone use that method if possible. Leaving this answer here as it does provide a working solution and demonstrates the use of the ng-required directive.

Improve this answer
Sign up to request clarification or add additional context in comments.

6 Comments

ng-required instead of just required does it. Thanks.
this solution doesn't work, not even in the provided plnkr demo.
@ConquerorsHaki It seems to be still working for me. You are allowed to submit the form when a value is selected. What are you seeing?
exuse me @dmullings I must have confused the plnkrs. It indeed does work like a charm!
Where is SaintScott's answer? Thanks!
|
98

I think what you need is to add a name for the radio group, for a radio input need a name to determine which it belongs to, the link below works validation without ng-required(the accepted answer)

<input type="radio" name='group' ng-model="color" value="red" required>  Red <br/>
<input type="radio" name='group' ng-model="color" value="green" required> Green <br/>
<input type="radio" name='group' ng-model="color" value="blue" required> Blue <br/>

http://plnkr.co/edit/LdgAywfC6mu7gL9SxPpC?p=preview

Improve this answer

4 Comments

Works fine, and the name of the group is necessary for correct radio-button working, particularly if you have several groups in a form...
@CilliéMalan, i guess it's always working this way. At least from 1.2.
Something important to note, is that a radio button group is considered valid regardless of an option being selected unless at least one of the radio buttons in a group has the ng-model attribute.
Another thing to note is that this doesn't fix the issue where one wants to bind on the $touched or $dirty property of ng-model, but there is a workaround (at least for $dirty) where ng-form can be nested and referred to. See similar question here - stackoverflow.com/questions/22181462/…
0

Alternative solution using a directive. Accepted answer didn't work for me in Firefox (v 33.0).

The idea was to set the required attribute to false on all radios with a specific class name, on change.

  • jQuery was added because I was having trouble with the jqlite remove attribute function.
  • I copied as much as possible from the original plunker.

http://plnkr.co/edit/nbzjQqCAvwNmkbLW5bxN?p=preview

<!doctype html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <title>Example - example-radio-input-directive-production</title>
  <script src="//code.jquery.com/jquery-1.11.0.min.js"></script>
  <script src="//ajax.googleapis.com/ajax/libs/angularjs/1.3.0-beta.14/angular.min.js"></script>
</head>
<body ng-app="radioExample">
  <script>
    angular.module('radioExample', [])
    .controller('ExampleController', ['$scope', function($scope) {
      $scope.myObject= {};

      $scope.specialValue = {
        "id": "12345",
        "value": "green"
      };

      $scope.submitForm = function() {
        alert('valid');
      }

    }])
    .directive('colorChoice', function() {
      return {
        restrict: 'E',
        template: '<div ng-repeat="c in colors"><input class="colorClass" type="radio" value={{c}} ng-model="myObject.color" required />{{c}}</div>',
        link: function(scope, element, attrs) {
          scope.colors = ['Red', 'Green', 'Blue'];

          element.on('change', function(){
            $(".colorClass").attr("required", false);
          });
        }
      }
    });
  </script>
  <form name="myForm" ng-submit="submitForm()" ng-controller="ExampleController">
    <color-choice></color-choice>
    <tt>color = {{myObject.color | json}}</tt><br/>
    <button type="submit">Submit</button>
  </form>

</body>
</html>
Improve this answer

3 Comments

If you have a new question, please ask it by clicking the Ask Question button. Include a link to this question if it helps provide context.
Sorry, this was supposed to be an alternative solution that works in Firefox (v 33.0). The currently accepted answer didn't work for me.
cool, so this is a solution that works? If so, awesome. Possibly just the language in your answer doesn't make that super-obvious :)
0

In my case, I was using angularJS material library and adding required attribute to <md-radio=group> tag did the trick.

Improve this answer

Comments

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.