Comment 1 for bug 2050227

Revision history for this message
Jason Stephenson (jstephenson) wrote (last edit ):

The working branch user/dyrcona/lp2050227-update-idl resolves this issue by adding the oils_obj:required attribute to required fields. "Required" is defined as a real field that exists in the database that can be updated (i.e. the object is not readonly and the field or object are not virtual), that has a "not null" constraint and lacks a default value.

The branch has 4 commits:

1. Adds only the required attribute.
2. Regularizes spacing in fields based on libxml2 writing out the IDL, and adjusting indentation based on the vim and Emacs settings in the file. (NB: There may be some whitespace "errors" that were missed.)
3. Fixes a typo and removes a duplicate permissions entry based on validating the result of the above with the fm_IDL.xsd schema.
4. Adds a release note to the "Architecture" section. (I struggled with putting it in Architecture or API. If you commit this and want to move the release note, I will NOT be fussed.)

URL for the branch is: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dyrcona/lp2050227-update-idl

I have lightly tested this by running the Perl tests and adding a few settings through the Angular admin interfaces. Everything that I've tried works.