From: Grégoire Aubert <gregoire.aubert@sonarsource.com>
Date: Wed, 7 Mar 2018 13:37:42 +0000 (+0100)
Subject: Revert "Merge branch 'master' of https://github.com/SonarSource/sonarqube"
X-Git-Tag: 7.5~1578
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=8b35325a410dd5b38ac2ee79ea1d1441d34e6de2;p=sonarqube.git

Revert "Merge branch 'master' of https://github.com/SonarSource/sonarqube"

This reverts commit ec739303a171b09089a1987d69777fa988b19eb3, reversing
changes made to da6a97014a1aba426ca6253623f6ff71281a64ac.
---

diff --git a/server/sonar-web/src/main/js/apps/code/bucket.ts b/server/sonar-web/src/main/js/apps/code/bucket.ts
index 7431dc36efa..8fc8bce8e65 100644
--- a/server/sonar-web/src/main/js/apps/code/bucket.ts
+++ b/server/sonar-web/src/main/js/apps/code/bucket.ts
@@ -17,7 +17,6 @@
  * along with this program; if not, write to the Free Software Foundation,
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
-import { concat } from 'lodash';
 import { Breadcrumb, Component } from './types';
 
 let bucket: { [key: string]: Component } = {};
@@ -44,10 +43,6 @@ export function addComponentChildren(
   total: number,
   page: number
 ): void {
-  const previous = getComponentChildren(componentKey);
-  if (previous) {
-    children = concat(previous.children, children);
-  }
   childrenBucket[componentKey] = { children, total, page };
 }
 
diff --git a/server/sonar-web/src/main/js/apps/code/components/__tests__/buckets-test.tsx b/server/sonar-web/src/main/js/apps/code/components/__tests__/buckets-test.tsx
deleted file mode 100644
index 49334c29a77..00000000000
--- a/server/sonar-web/src/main/js/apps/code/components/__tests__/buckets-test.tsx
+++ /dev/null
@@ -1,65 +0,0 @@
-/*
- * SonarQube
- * Copyright (C) 2009-2018 SonarSource SA
- * mailto:info AT sonarsource DOT com
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 3 of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
- */
-import { concat } from 'lodash';
-import { Component } from '../../types';
-import {
-  addComponent,
-  getComponent,
-  addComponentChildren,
-  getComponentChildren
-} from '../../bucket';
-
-const component: Component = { key: 'frodo', name: 'frodo', qualifier: 'frodo' };
-
-const componentKey: string = 'foo';
-const childrenA: Component[] = [
-  { key: 'foo', name: 'foo', qualifier: 'foo' },
-  { key: 'bar', name: 'bar', qualifier: 'bar' }
-];
-const childrenB: Component[] = [
-  { key: 'bart', name: 'bart', qualifier: 'bart' },
-  { key: 'simpson', name: 'simpson', qualifier: 'simpson' }
-];
-
-it('Should have empty bucket at start', () => {
-  expect(getComponent(component.key)).toBeUndefined();
-});
-
-it('Should be able to store components in a bucket', () => {
-  addComponent(component);
-  expect(getComponent(component.key)).toEqual(component);
-});
-
-it('Should have empty children bucket at start', () => {
-  expect(getComponentChildren(componentKey)).toBeUndefined();
-});
-
-it('Should be able to store children components in a bucket', () => {
-  addComponentChildren(componentKey, childrenA, childrenA.length, 1);
-  expect(getComponentChildren(componentKey).children).toEqual(childrenA);
-});
-
-it('Should append new children components at the end of the bucket', () => {
-  addComponentChildren(componentKey, childrenB, 4, 2);
-  const finalBucket = getComponentChildren(componentKey);
-  expect(finalBucket.children).toEqual(concat(childrenA, childrenB));
-  expect(finalBucket.total).toBe(4);
-  expect(finalBucket.page).toBe(2);
-});
diff --git a/server/sonar-web/src/main/js/components/workspace/models/items.js b/server/sonar-web/src/main/js/components/workspace/models/items.js
index 0b1c4c3adf1..c191ca98a90 100644
--- a/server/sonar-web/src/main/js/components/workspace/models/items.js
+++ b/server/sonar-web/src/main/js/components/workspace/models/items.js
@@ -19,7 +19,6 @@
  */
 import Backbone from 'backbone';
 import Item from './item';
-import { saveWorkspace, getWorkspace } from '../../../helpers/storage';
 
 const STORAGE_KEY = 'sonarqube-workspace';
 
@@ -32,11 +31,11 @@ export default Backbone.Collection.extend({
 
   save() {
     const dump = JSON.stringify(this.toJSON());
-    saveWorkspace(dump);
+    window.localStorage.setItem(STORAGE_KEY, dump);
   },
 
   load() {
-    const dump = getWorkspace(STORAGE_KEY);
+    const dump = window.localStorage.getItem(STORAGE_KEY);
     if (dump != null) {
       try {
         const parsed = JSON.parse(dump);
diff --git a/server/sonar-web/src/main/js/helpers/storage.ts b/server/sonar-web/src/main/js/helpers/storage.ts
index 37533dbdfaa..fc54899c8b4 100644
--- a/server/sonar-web/src/main/js/helpers/storage.ts
+++ b/server/sonar-web/src/main/js/helpers/storage.ts
@@ -28,8 +28,6 @@ const PROJECTS_SORT = 'sonarqube.projects.sort';
 const PROJECT_ACTIVITY_GRAPH = 'sonarqube.project_activity.graph';
 const PROJECT_ACTIVITY_GRAPH_CUSTOM = 'sonarqube.project_activity.graph.custom';
 
-const WORKSPACE = 'sonarqube-workspace';
-
 function save(key: string, value?: string): void {
   try {
     if (value) {
@@ -43,14 +41,6 @@ function save(key: string, value?: string): void {
   }
 }
 
-export function saveWorkspace(dump: string): void {
-  window.localStorage.setItem(WORKSPACE, dump);
-}
-
-export function getWorkspace(): string | null {
-  return window.localStorage.getItem(WORKSPACE);
-}
-
 export function saveFavorite(): void {
   save(PROJECTS_DEFAULT_FILTER, PROJECTS_FAVORITE);
 }