On 11/5/2010 6:52 PM, Adam Young wrote:
Here it is.

ACK-ish.. :)

The error handling on ticket expiration works, but there are some minor issues:

1. The IPA variable is declared twice.

+var IPA;
 var IPA = ( function () {

2. IPA_DEFAULT_JSON_URL should be deleted.

 var IPA_DEFAULT_JSON_URL = '/ipa/json';
+    var default_json_url = '/ipa/json';

3. Is it necessary to call hasOwnProperty() in this loop?
   Variable "a" is obtained from attribute_members, so
   I think the hasOwnProperty() will always return true.

     for (var a in attribute_members) {
+        if (attribute_members.hasOwnProperty(a)){

If you could fix this before pushing that would be great. Thanks!

--
Endi S. Dewata

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to