1 files changed, 70 insertions, 69 deletions
diff --git a/Swiften/Parser/PayloadParsers/FormParser.cpp b/Swiften/Parser/PayloadParsers/FormParser.cpp
index 2df0a85..7c29189 100644
--- a/Swiften/Parser/PayloadParsers/FormParser.cpp
+++ b/Swiften/Parser/PayloadParsers/FormParser.cpp
@@ -1,4 +1,4 @@
 /*
- * Copyright (c) 2010 Remko Tronçon
+ * Copyright (c) 2010-2013 Remko Tronçon
  * Licensed under the GNU General Public License v3.
  * See Documentation/Licenses/GPLv3.txt for more information.
@@ -11,5 +11,5 @@
 namespace Swift {
 
-FormParser::FormParser() : level_(TopLevel), parsingItem_(false), parsingReported_(false) {
+FormParser::FormParser() : level_(TopLevel), parsingItem_(false), parsingReported_(false), parsingOption_(false) {
 }
 
@@ -37,65 +37,60 @@ void FormParser::handleStartElement(const std::string& element, const std::strin
 			currentText_.clear();
 		}
-		else if (element == "field") {
-			std::string type;
-			FormField::ref correspondingReportedField;
-			if (!parsingItem_) {
-				type = attributes.getAttribute("type");
-			} else {
-				foreach(FormField::ref field, getPayloadInternal()->getReportedFields()) {
-					if (field->getName() == attributes.getAttribute("var")) {
-						correspondingReportedField = field;
-						break;
+		else if (element == "reported") {
+			parsingReported_ = true;
 		}
+		else if (element == "item") {
+			parsingItem_ = true;
 		}
 	}
-			if (type == "boolean" || boost::dynamic_pointer_cast<BooleanFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = BooleanFormFieldParseHelper::create();
+	else if (level_ == FieldLevel && currentField_) {
+		currentText_.clear();
+		if (element == "option") {
+			currentOptionLabel_ = attributes.getAttribute("label");
+			currentOptionValue_ = "";
+			parsingOption_ = true;
 		}
-			else if (type == "fixed" || boost::dynamic_pointer_cast<FixedFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = FixedFormFieldParseHelper::create();
 	}
-			else if (type == "hidden" || boost::dynamic_pointer_cast<HiddenFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = HiddenFormFieldParseHelper::create();
+	if (level_ >= PayloadLevel) {
+		if (element == "field") {
+			currentField_ = boost::make_shared<FormField>();
+			std::string type = attributes.getAttribute("type");
+			FormField::Type fieldType = FormField::UnknownType;
+			if (type == "boolean") {
+				fieldType = FormField::BooleanType; 
 			}
-			else if (type == "jid-multi" || boost::dynamic_pointer_cast<JIDMultiFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = JIDMultiFormFieldParseHelper::create();
+			if (type == "fixed") {
+				fieldType = FormField::FixedType; 
 			}
-			else if (type == "jid-single" || boost::dynamic_pointer_cast<JIDSingleFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = JIDSingleFormFieldParseHelper::create();
+			if (type == "hidden") {
+				fieldType = FormField::HiddenType; 
 			}
-			else if (type == "list-multi" || boost::dynamic_pointer_cast<ListMultiFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = ListMultiFormFieldParseHelper::create();
+			if (type == "list-single") {
+				fieldType = FormField::ListSingleType; 
 			}
-			else if (type == "list-single" || boost::dynamic_pointer_cast<ListSingleFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = ListSingleFormFieldParseHelper::create();
+			if (type == "text-multi") {
+				fieldType = FormField::TextMultiType; 
 			}
-			else if (type == "text-multi" || boost::dynamic_pointer_cast<TextMultiFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = TextMultiFormFieldParseHelper::create();
+			if (type == "text-private") {
+				fieldType = FormField::TextPrivateType; 
 			}
-			else if (type == "text-private" || boost::dynamic_pointer_cast<TextPrivateFormField>(correspondingReportedField)) {
-				currentFieldParseHelper_ = TextPrivateFormFieldParseHelper::create();
+			if (type == "text-single") {
+				fieldType = FormField::TextSingleType; 
 			}
-			else /*if (type == "text-single") || undefined */ {
-				currentFieldParseHelper_ = TextSingleFormFieldParseHelper::create();
+			if (type == "jid-single") {
+				fieldType = FormField::JIDSingleType; 
 			}
-			if (currentFieldParseHelper_) {
-				currentFieldParseHelper_->getField()->setName(attributes.getAttribute("var"));
-				currentFieldParseHelper_->getField()->setLabel(attributes.getAttribute("label"));
+			if (type == "jid-multi") {
+				fieldType = FormField::JIDMultiType; 
 			}
+			if (type == "list-multi") {
+				fieldType = FormField::ListMultiType; 
 			}
-		else if (element == "reported") {
-			parsingReported_ = true;
-			level_ = PayloadLevel - 1;
+			currentField_->setType(fieldType);
+			currentField_->setName(attributes.getAttribute("var"));
+			currentField_->setLabel(attributes.getAttribute("label"));
 		}
-		else if (element == "item") {
-			parsingItem_ = true;
-			level_ = PayloadLevel - 1;
-		}
-	}
-	else if (level_ == FieldLevel && currentFieldParseHelper_) {
+		else if (element == "value") {
 			currentText_.clear();
-		if (element == "option") {
-			currentOptionLabel_ = attributes.getAttribute("label");
 		}
 	}
@@ -124,40 +119,46 @@ void FormParser::handleEndElement(const std::string& element, const std::string&
 			}
 		}
-		else if (element == "field") {
-			if (currentFieldParseHelper_) {
-				if (parsingReported_) {
-					getPayloadInternal()->addReportedField(currentFieldParseHelper_->getField());
-				} else if (parsingItem_) {
-					currentFields_.push_back(currentFieldParseHelper_->getField());
-				} else {
-					getPayloadInternal()->addField(currentFieldParseHelper_->getField());
-				}
-				currentFieldParseHelper_.reset();
+		else if (element == "reported") {
+			parsingReported_ = false;
 		}
+		else if (element == "item") {
+			parsingItem_ = false;
+			getPayloadInternal()->addItem(currentFields_);
+			currentFields_.clear();
 		}
 	}
-	else if (level_ == FieldLevel && currentFieldParseHelper_) {
+	else if (currentField_) {
 		if (element == "required") {
-			currentFieldParseHelper_->getField()->setRequired(true);
+			currentField_->setRequired(true);
 		}
 		else if (element == "desc") {
-			currentFieldParseHelper_->getField()->setDescription(currentText_);
+			currentField_->setDescription(currentText_);
 		}
 		else if (element == "option") {
-			currentFieldParseHelper_->getField()->addOption(FormField::Option(currentOptionLabel_, currentText_));
+			currentField_->addOption(FormField::Option(currentOptionLabel_, currentOptionValue_));
+			parsingOption_ = false;
 		}
 		else if (element == "value") {
-			currentFieldParseHelper_->addValue(currentText_);
+			if (parsingOption_) {
+				currentOptionValue_ = currentText_;
 			}
+			else {
+				currentField_->addValue(currentText_);
 			}
-	if (element == "reported") {
-		parsingReported_ = false;
-		level_++;
 		}
-	else if (element == "item") {
-		parsingItem_ = false;
-		level_++;
-		getPayloadInternal()->addItem(currentFields_);
-		currentFields_.clear();
+	}
+	if (level_ >= PayloadLevel && currentField_) {
+		if (element == "field") {
+			if (parsingReported_) {
+				getPayloadInternal()->addReportedField(currentField_);
+			} 
+			else if (parsingItem_) {
+				currentFields_.push_back(currentField_);
+			} 
+			else {
+				getPayloadInternal()->addField(currentField_);
+			}
+			currentField_.reset();
+		}
 	}
 }