SONAR-7288 Use api/favorites/remove in the UI

server/sonar-web/src/main/js/api/favorites.js

@@ -18,17 +18,16 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 /* @flow */
-import { post, requestDelete, getJSON } from '../helpers/request';
+import { post, getJSON } from '../helpers/request';
 
 export const getFavorites = (): Promise<Object> => (
     getJSON('/api/favourites')
 );
 
 export function addFavorite (component: string) {
-  return post('/api/favorites/add', { component});
+  return post('/api/favorites/add', { component });
 }
 
-export function removeFavorite (componentKey: string) {
-  const url = '/api/favourites/' + encodeURIComponent(componentKey);
-  return requestDelete(url);
+export function removeFavorite (component: string) {
+  return post('/api/favorites/remove', { component });
 }

server/sonar-web/src/main/js/components/source-viewer/header.js

@@ -25,7 +25,6 @@ import MoreActionsView from './more-actions';
 import MeasuresOverlay from './measures-overlay';
 import Template from './templates/source-viewer-header.hbs';
 
-const API_FAVOURITE: string = window.baseUrl + '/api/favourites';
 const API_FAVORITE: string = window.baseUrl + '/api/favorites';
 
 export default Marionette.ItemView.extend({
@@ -43,8 +42,11 @@ export default Marionette.ItemView.extend({
     const that = this;
     if (this.model.get('fav')) {
       $.ajax({
-        url: API_FAVOURITE + '/' + this.model.get('key'),
-        type: 'DELETE'
+        url: API_FAVORITE + '/remove',
+        type: 'POST',
+        data: {
+          component: this.model.get('key')
+        }
       }).done(function () {
         that.model.set('fav', false);
         that.render();

server/sonar-web/src/main/webapp/WEB-INF/app/controllers/api/favourites_controller.rb

@@ -36,15 +36,6 @@ class Api::FavouritesController < Api::ApiController
     end
   end
 
-  #
-  # DELETE /api/favourites/<key>
-  # curl -X DELETE  http://localhost:9000/api/favourites/org.apache.struts:struts -v -u admin:admin
-  #
-  def destroy
-    ok=current_user.delete_favourite(params[:id])
-    render_success(ok ? "Favourite deleted" : "Favourite not found")
-  end
-
   def favourites_to_json(favourites=[])
     json=[]
     favourites.each do |f|