[Ovirt-devel] [PATCH server] network integration into ovirt server db and wui

[Darryl Pierce]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Mohammed Morsi wrote:
>> Why not just use validates_presence_of for these? This seems like the
>> standard out-of-the-box use case for validates_presence_of. Also, it
>> looks like name and interface_name have the valiadition in both places
>> -- validates_presence_of and validate.
> See my comments to Darryl's response to my patch. I am just concerned
> the 'validates_presence_of' doesn't add anything to the errors array
> which is needed by the wui. If this concern is unfound, then I can
> remove the validations and simply use validates_presence_of

It should be adding the errors there, definitely. Look at the source in
the link I gave you earlier and you'll see where it does add those error
messages to the record.error attribute.

- --
Darryl L. Pierce <dpierce at redhat.com> : GPG KEYID: 6C4E7F1B
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iEYEARECAAYFAkkLKc8ACgkQjaT4DmxOfxsSvwCdEoWtgeE8o0g/UCFlR6cCQIgq
TG4AmwR7GvpkSIqUyGbW+mOqwxhce+gr
=EzYd
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dpierce.vcf
Type: text/x-vcard
Size: 319 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/ovirt-devel/attachments/20081031/2d0dc325/attachment.vcf>