Skip to content

Commit fe861eb

Browse files
committed
Don't overwrite params in queryParams
1 parent b1e4b7d commit fe861eb

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

TinCan/RemoteLRS.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ limitations under the License.
2222
using Newtonsoft.Json.Linq;
2323
using TinCan.Documents;
2424
using TinCan.LRSResponses;
25+
using System.Linq;
2526

2627
namespace TinCan
2728
{
@@ -81,7 +82,7 @@ public MyHTTPResponse(HttpWebResponse webResp)
8182
}
8283
}
8384

84-
private string AppendParamsToExistingQueryString(string currentQueryString, Dictionary<string, string> parameters)
85+
private string AppendParamsToExistingQueryString(string currentQueryString, IEnumerable<KeyValuePair<string, string>> parameters)
8586
{
8687
foreach (KeyValuePair<String, String> entry in parameters)
8788
{
@@ -113,7 +114,7 @@ private MyHTTPResponse MakeSyncRequest(MyHTTPRequest req)
113114

114115
String qs = "";
115116
qs = AppendParamsToExistingQueryString(qs, req.queryParams);
116-
qs = AppendParamsToExistingQueryString(qs, extended);
117+
qs = AppendParamsToExistingQueryString(qs, extended.Where(w => !req.queryParams.ContainsKey(w.Key)));
117118

118119
if (qs != "")
119120
{

0 commit comments

Comments
 (0)