correctly handle multiple instances of the same command on a slide

rstudio · Feb 15, 2013 · 3b0cb96 · 3b0cb96
1 parent fe823cb
commit 3b0cb96
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/cpp/session/modules/presentation/SlideParser.cpp b/src/cpp/session/modules/presentation/SlideParser.cpp
@@ -120,12 +120,10 @@ bool Slide::showTitle() const
 std::vector<Command> Slide::commands() const
 {
    std::vector<Command> commands;
-   std::vector<std::string> flds = fields();
-   for (size_t i=0; i<flds.size(); i++)
+   BOOST_FOREACH(const Slide::Field& field, fields_)
    {
-      std::string field = flds[i];
-      if (isCommandField(field))
-         commands.push_back(Command(field, fieldValue(field)));
+      if (isCommandField(field.first))
+         commands.push_back(Command(field.first, field.second));
    }
    return commands;
 }