added a comment -
01/Mar/10 08:17 AM Comments from Megan
I looked into this again and uncovered a couple of bugs very quickly! However, they're unrelated. The problem for this issue, is that when the metadata is retrieved it doesn't include the conditional metadata, in the code it says - "/* this is not implemented...*/". But when it updates the metadata it does all fields, since the conditional fieldset is empty it writes null values in.
So there are 3 issues here:
1. get_metadata does not return the conditional metadata.
2. update_metadata updates all fields / fieldsets and doesn't skip the ones that aren't included.
3. update_metadata writes "n/a" in for null values if they were empty in the get_metadata query.
All 3 are bugs which will need to be logged and pulled into our sprints.
I looked into this again and uncovered a couple of bugs very quickly! However, they're unrelated. The problem for this issue, is that when the metadata is retrieved it doesn't include the conditional metadata, in the code it says - "/* this is not implemented...*/". But when it updates the metadata it does all fields, since the conditional fieldset is empty it writes null values in.
So there are 3 issues here:
1. get_metadata does not return the conditional metadata.
2. update_metadata updates all fields / fieldsets and doesn't skip the ones that aren't included.
3. update_metadata writes "n/a" in for null values if they were empty in the get_metadata query.
All 3 are bugs which will need to be logged and pulled into our sprints.