From: Chocobozzz <me@florianbigard.com>
Date: Fri, 26 Jun 2020 14:01:04 +0000 (+0200)
Subject: Fix query string parsing
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=cc0e0d32ea7f65475630bcdee1f1fa8a8c71eb05;p=oweals%2Fpeertube.git

Fix query string parsing
---

diff --git a/client/src/app/+admin/moderation/video-abuse-list/video-abuse-details.component.html b/client/src/app/+admin/moderation/video-abuse-list/video-abuse-details.component.html
index 5512bb1de..ec808cdb8 100644
--- a/client/src/app/+admin/moderation/video-abuse-list/video-abuse-details.component.html
+++ b/client/src/app/+admin/moderation/video-abuse-list/video-abuse-details.component.html
@@ -17,7 +17,7 @@
             <span class="text-muted">{{ videoAbuse.reporterAccount.nameWithHost }}</span>
           </div>
         </a>
-        <a [routerLink]="[ '/admin/moderation/video-abuses/list' ]" [queryParams]="{ 'search': 'reportee:&quot;' + videoAbuse.reporterAccount.displayName + '&quot;' }" class="ml-auto text-muted video-details-links" i18n>
+        <a [routerLink]="[ '/admin/moderation/video-abuses/list' ]" [queryParams]="{ 'search': 'reporter:&quot;' + videoAbuse.reporterAccount.displayName + '&quot;' }" class="ml-auto text-muted video-details-links" i18n>
           {videoAbuse.countReportsForReporter, plural, =1 {1 report} other {{{ videoAbuse.countReportsForReporter }} reports}}<span class="ml-1 glyphicon glyphicon-flag"></span>
         </a>
       </span>
@@ -90,4 +90,4 @@
       <div *ngIf="!videoAbuse.video.deleted && !videoAbuse.video.blacklisted" [innerHTML]="videoAbuse.embedHtml"></div>
     </div>
   </div>
-</div>
\ No newline at end of file
+</div>
diff --git a/client/src/app/core/rest/rest.service.ts b/client/src/app/core/rest/rest.service.ts
index 78558851a..c12b6bd41 100644
--- a/client/src/app/core/rest/rest.service.ts
+++ b/client/src/app/core/rest/rest.service.ts
@@ -68,8 +68,9 @@ export class RestService {
   parseQueryStringFilter (q: string, prefixes: QueryStringFilterPrefixes): ParseQueryStringFilterResult {
     if (!q) return {}
 
-    // Tokenize the strings using spaces
-    const tokens = q.split(' ').filter(token => !!token)
+    // Tokenize the strings using spaces that are not in quotes
+    const tokens = q.match(/(?:[^\s"]+|"[^"]*")+/g)
+                    .filter(token => !!token)
 
     // Build prefix array
     const prefixeStrings = Object.values(prefixes)
@@ -88,6 +89,7 @@ export class RestService {
 
       const matchedTokens = tokens.filter(t => t.startsWith(prefix))
                                   .map(t => t.slice(prefix.length)) // Keep the value filter
+                                  .map(t => t.replace(/^"|"$/g, ''))
                                   .map(t => {
                                     if (prefixObj.handler) return prefixObj.handler(t)