diff options
author | Alex Clayton <alex.clayton@isode.com> | 2016-03-21 16:22:56 (GMT) |
---|---|---|
committer | Alex Clayton <alex.clayton@isode.com> | 2016-03-22 11:32:44 (GMT) |
commit | 9b518dcdd98d694a4464895fce17a3c7a8caf569 (patch) | |
tree | f585f40f6ac0c0d829015c68d6d8c8169a4fb252 /src/com/isode/stroke/parser/payloadparsers | |
parent | 0ec7a2d8ab803b0e2df2c616f61d2b69b042561f (diff) | |
download | stroke-9b518dcdd98d694a4464895fce17a3c7a8caf569.zip stroke-9b518dcdd98d694a4464895fce17a3c7a8caf569.tar.bz2 |
Tidy XEP-0141 code
As per swiften patch of the same name.
This change addresses some feedback the previous XEP-0141 commit.
Test-information: Unit tests stil pass.
Change-Id: I3a9f9ec93b56352418f8288371bbd6874e01310d
Diffstat (limited to 'src/com/isode/stroke/parser/payloadparsers')
-rw-r--r-- | src/com/isode/stroke/parser/payloadparsers/FormParser.java | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/com/isode/stroke/parser/payloadparsers/FormParser.java b/src/com/isode/stroke/parser/payloadparsers/FormParser.java index ab14a4e..bbb04c0 100644 --- a/src/com/isode/stroke/parser/payloadparsers/FormParser.java +++ b/src/com/isode/stroke/parser/payloadparsers/FormParser.java @@ -38,7 +38,6 @@ public class FormParser extends GenericPayloadParser<Form> { private String currentText_ = ""; private String currentFieldRef_ = ""; private boolean parsingItem_ = false; - private boolean parseStarted_ = false; private boolean hasReportedRef_ = false; private FormText currentTextElement_; private FormReportedRef currentReportedRef_; @@ -242,15 +241,15 @@ public class FormParser extends GenericPayloadParser<Form> { else { if (currentPages_.size() > 0) { for (FormPage page : currentPages_) { - for (String pRef : page.getFieldRefs()) { - if (pRef.equals(currentField_.getName())) { + for (String pageRef : page.getFieldRefs()) { + if (pageRef.equals(currentField_.getName())) { page.addField(currentField_); } } } for (FormSection section : currentSections_) { - for (String sRef : section.getFieldRefs()) { - if (sRef.equals(currentField_.getName())) { + for (String sectionRef : section.getFieldRefs()) { + if (sectionRef.equals(currentField_.getName())) { section.addField(currentField_); } } @@ -270,7 +269,7 @@ public class FormParser extends GenericPayloadParser<Form> { sectionStack_.remove(sectionStack_.size()-1); } else if (sectionStack_.size() == 1) { - // Add the remaining section on the stack to it's parent page + // Add the remaining section on the stack to its parent page currentPage_.addChildSection(sectionStack_.get(sectionStack_.size()-1)); sectionStack_.remove(sectionStack_.size()-1); } |