Changes

Jump to: navigation, search

JSON Integration Adapter Code Review 1 Changes

495 bytes removed, 15:20, 10 August 2011
JSONMessagePartMapping
{| border="1" cellpadding="4" cellspacing="2" style="border: 1px solid black;border-collapse:collapse;"
|-
! Issue! DescriptionTask
! Status
|-
| Use Hashholder instead of HashSet
| Use HashHolder in the XMLJSONMessagePartLoader instead of HashSet and whenever appropriate replace HashSet with HashHolder
|
|-
| Change 0x000 for byteinitialization| For root modes, change 0xXXX to a cast to byte. Also do not use * Remove leading zeroes.zeros
|
|-
| StringParser* Remove StringParsre from JSONMessagePartMapping| Either remove StringParser from the mapping, or use it in formatting primitives. Do not leave it in if it is not used.needed
|
|-
| Create class called RootJSONMessagePartMapping| * Create a class called RootJSONMessagePartMapping for each CompositeMessagePart. This * Extend JSONMessagePartMapping and move all root JSON mapping will handle the "mode" for each CompositeMessagePartsettings to this.
|
|-
| Allow root Composite Message Part modes for each CompositeMessagePart| * Rename root mode to mode, so that each CompositeMessagePart can use every type of * Allow all composite message parts to have a mode.
|
|-
| Rename m_sFormat| Rename the variable m_sFormat * Remove if not used (or use it and keep the name).needed
|
|}
 
===== JSONMessageParser =====
1
edit

Navigation menu