Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix227 224 251 #29

Merged
merged 2 commits into from Mar 29, 2012

Conversation

Projects
None yet
2 participants
Contributor

jcookems commented Mar 26, 2012

Fixes the following issues:

  • #224: Table: Cannot insert entity with null property values
  • #227: Table: XML-valued string properties not deserialized correctly
  • #251: Table: Entity properties with empty strings need to be supported

@aliakb aliakb and 1 other commented on an outdated diff Mar 26, 2012

...e/services/table/implementation/AtomReaderWriter.java
Object value = edmValueConverter.deserialize(edmType, serializedValue);
result.put(name, new Property().setEdmType(edmType).setValue(value));
- nextSignificant(xmlr);
+ if (!xmlr.isEndElement()) {
@aliakb

aliakb Mar 26, 2012

The condition inside "if" will never evaluate to true, the "while" loop above it guarantees that.

@jcookems

jcookems Mar 29, 2012

Contributor

Ah, yes. That was a left over from the first attempt for a fix. I've removed the "if" statement.

jcookems added a commit that referenced this pull request Mar 29, 2012

@jcookems jcookems merged commit 46d807c into Azure:dev-bookmark Mar 29, 2012

joostdenijs pushed a commit to joostdenijs/azure-sdk-for-java that referenced this pull request Jan 18, 2013

Merge pull request #29 from christav/list-job-assets-508
Can now list input and output assets for jobs

joostdenijs pushed a commit to joostdenijs/azure-sdk-for-java that referenced this pull request Jan 18, 2013

gcheng pushed a commit that referenced this pull request Mar 27, 2013

Merge pull request #29 from gcheng/nimbusfailure
fix unit test failures due to media service upgrade.

andrerod pushed a commit that referenced this pull request May 9, 2014

Merge pull request #29 from andrerod/dev_4
Fix braces and enabling checkstyle on build

jianghaolu pushed a commit to jianghaolu/azure-sdk-for-java that referenced this pull request Apr 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment