Merge pull request #1196 from data-for-change/1195-modify-patch

modify patch request

Author: atalyaalon

src/services/news.data.service.ts

@@ -68,20 +68,27 @@ function onErrorFetchNewsFlash() {
   return errorArr;
 }
 
-export function updateNews(newsId: number, newLocationQualification: any,
-                    streetLocation: IStreetData | null, gpsLocation: IGpsData | null) {
-  const data = [];
-  data.push(`newsflash_location_qualification=${newLocationQualification}`)
+export function updateNews(
+  newsId: number,
+  newLocationQualification: any,
+  streetLocation: IStreetData | null,
+  gpsLocation: IGpsData | null
+) {
+  const payload: any = {
+    newsflash_location_qualification: newLocationQualification,
+  };
+
   if (gpsLocation) {
-    data.push(`road_segment_id=${gpsLocation.road_segment_id}`)
-    data.push(`road1=${gpsLocation.road1}`)
+    payload.road_segment_id = gpsLocation.road_segment_id;
+    payload.road1 = gpsLocation.road1;
   } else if (streetLocation) {
-    data.push(`yishuv_name=${streetLocation.city.yishuv_name}`)
-    data.push(`street1_hebrew=${streetLocation.street.street_hebrew}`)
+    payload.yishuv_name = streetLocation.city.yishuv_name;
+    payload.street1_hebrew = streetLocation.street.street_hebrew;
   }
-  const url = `${NEWS_FLASH_API}/${newsId}?${data.join('&')}`;
-  axios
-    .patch(url, {}, { withCredentials: true })
+
+  const url = `${NEWS_FLASH_API}/${newsId}`;
+  return axios
+    .patch(url, payload, { withCredentials: true })
     .then((res) => res.data)
-    .catch(onErrorFetchNewsFlash)
-}
+    .catch(onErrorFetchNewsFlash);
+}