bool ItemListHandler::ParseItemElement(parser::DictionaryValue* item_element,
std::shared_ptr<ItemInfo> info_output,
- std::string* /*error*/) {
+ std::string* error) {
std::string item_value, item_type;
if (item_element->GetString(std::string(attribute) + attribute_name,
- &item_type))
+ &item_type)) {
info_output->set_type(item_type);
- if (item_element->GetString(value, &item_value))
+ } else {
+ *error = std::string("Can't access ") + attribute_name + " attribute.";
+ return false;
+ }
+ if (item_element->GetString(value, &item_value)) {
info_output->set_value(item_value);
+ } else {
+ *error = std::string("Can't access value of") + attribute_name +
+ " attribute.";
+ return false;
+ }
return true;
}