From 6316b9b7aed7d67c585f8c17cce5669f8cbb5d65 Mon Sep 17 00:00:00 2001
From: Enrico Forestieri <forenr@lyx.org>
Date: Mon, 4 Feb 2019 10:49:02 +0100
Subject: [PATCH] Fix bug #11484

When splitting parameters at commas, take into account that
commas inside curly braces are not parameter separators.
---
 src/insets/InsetInclude.cpp | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/src/insets/InsetInclude.cpp b/src/insets/InsetInclude.cpp
index 8d1a4a87c5..121b1ad8a9 100644
--- a/src/insets/InsetInclude.cpp
+++ b/src/insets/InsetInclude.cpp
@@ -165,6 +165,23 @@ InsetLabel * createLabel(Buffer * buf, docstring const & label_str)
 	return new InsetLabel(buf, icp);
 }
 
+
+char_type replaceCommaInBraces(docstring & params)
+{
+	// Code point from private use area
+	char_type private_char = 0xE000;
+	int count = 0;
+	for (char_type & c : params) {
+		if (c == '{')
+			++count;
+		else if (c == '}')
+			--count;
+		else if (c == ',' && count)
+			c = private_char;
+	}
+	return private_char;
+}
+
 } // namespace
 
 
@@ -625,12 +642,18 @@ void InsetInclude::latex(otexstream & os, OutputParams const & runparams) const
 		docstring label;
 		docstring placement;
 		bool isfloat = lstparams.isFloat();
+		// We are going to split parameters at commas, so
+		// replace commas that are not parameter separators
+		// with a code point from the private use area
+		char_type comma = replaceCommaInBraces(parameters);
 		// Get float placement, language, caption, and
 		// label, then remove the relative options if minted.
 		vector<docstring> opts =
 			getVectorFromString(parameters, from_ascii(","), false);
 		vector<docstring> latexed_opts;
 		for (size_t i = 0; i < opts.size(); ++i) {
+			// Restore replaced commas
+			opts[i] = subst(opts[i], comma, ',');
 			if (use_minted && prefixIs(opts[i], from_ascii("float"))) {
 				if (prefixIs(opts[i], from_ascii("float=")))
 					placement = opts[i].substr(6);