`bcfg -ven` vs. `bcfg -I` yield different output

[https://www.google.com/accounts/o8/id?id=AItOawkjKgZsrk-wj8Ikb2eLLRqMctAjiqUuv2g]

When using the bcfg2-accounts plugin (​https://github.com/weiss/bcfg2-accounts),\

and adding a new user to Properties/accounts.xml.

bcfg2 -n : gives no indication that an action is necessary.

bcfg2 -I : works as expected. Expected meaning you are prompted to modify the \ user.

An example can be found here: ​http://pastebin.com/A0Uf4Gz4. You may ignore the\

question about why usermod is being called at all.

[https://profiles.google.com/holger.weiss]

Just to clarify: this issue is unrelated to the bcfg2-accounts stuff. It can easily be reproduced using the following entry as a test case:

<Action name='test' timing='pre' when='always' status='check' command='/bin/true'/>

This <Action> won't be mentioned by bcfg2 -vn, and it won't be added to the list of Bad Entries. If the timing attribute is set to post, the entry will be handled as expected.

[https://www.google.com/accounts/o8/id?id=AItOawnSjgovXZr-_V3vGkvMSR0pc5LDykRc1Nc]

Fixed in ​https://github.com/Bcfg2/bcfg2/commit/addb374b3388c7faa33ff5664293348874dd295d

[https://www.google.com/accounts/o8/id?id=AItOawlP6DRBvIJ7NLpLWILbgzURvGbMtGEyd8U]

I can reproduce the problem with Bcfg2 1.2.3.

[https://www.google.com/accounts/o8/id?id=AItOawnSjgovXZr-_V3vGkvMSR0pc5LDykRc1Nc]

Sorry, I misunderstood the issue here; pre and both actions were not reliably showing up in -I, either, and that's what I fixed.

I've now fixed the issue with -n output as well:

​https://github.com/Bcfg2/bcfg2/commit/a95c365f39d983986e1a69e8396101e87ff08abb ​https://github.com/Bcfg2/bcfg2/commit/a7c9889a88233f8ea3ff7f05fa4234d30cf9279c