// Copyright (C) 2016 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

package com.google.gerrit.server.patch;

import com.google.gerrit.entities.Patch;
import com.google.gerrit.entities.Project;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.Callable;

public class DiffSummaryLoader implements Callable<DiffSummary> {
  public interface Factory {
    DiffSummaryLoader create(DiffSummaryKey key, Project.NameKey project);
  }

  private final PatchListCache patchListCache;
  private final DiffSummaryKey key;
  private final Project.NameKey project;

  @Inject
  DiffSummaryLoader(PatchListCache plc, @Assisted DiffSummaryKey k, @Assisted Project.NameKey p) {
    patchListCache = plc;
    key = k;
    project = p;
  }

  @Override
  public DiffSummary call() throws Exception {
    PatchList patchList = patchListCache.get(key.toPatchListKey(), project);
    return toDiffSummary(patchList);
  }

  private DiffSummary toDiffSummary(PatchList patchList) {
    List<String> r = new ArrayList<>(patchList.getPatches().size());
    for (PatchListEntry e : patchList.getPatches()) {
      if (Patch.isMagic(e.getNewName())) {
        continue;
      }
      switch (e.getChangeType()) {
        case ADDED:
        case MODIFIED:
        case DELETED:
        case COPIED:
        case REWRITE:
          r.add(e.getNewName());
          break;

        case RENAMED:
          r.add(e.getOldName());
          r.add(e.getNewName());
          break;
      }
    }
    return new DiffSummary(
        r.stream().sorted().toArray(String[]::new),
        patchList.getInsertions(),
        patchList.getDeletions());
  }
}

@@ -723,24 +723,87 @@
 es.Project;%0A
+import com.google.gerrit.server.patch.filediff.FileDiffOutput;%0A
 import com.g
@@ -920,24 +920,46 @@
 .util.List;%0A
+import java.util.Map;%0A
 import java.
@@ -983,16 +983,54 @@
 allable;
+%0Aimport org.eclipse.jgit.lib.ObjectId;
 %0A%0Apublic
@@ -1218,37 +1218,37 @@
 nal 
-PatchListCache patchListCache
+DiffOperations diffOperations
 ;%0A  
@@ -1357,26 +1357,37 @@
 der(
-PatchListCache plc
+%0A      DiffOperations diffOps
 , @A
@@ -1451,28 +1451,32 @@
     
-patchListCache = plc
+diffOperations = diffOps
 ;%0A  
@@ -1575,78 +1575,341 @@
     
-PatchList patchList = patchListCache.get(key.toPatchListKey(), project
+ObjectId oldId = key.toPatchListKey().getOldId();%0A    ObjectId newId = key.toPatchListKey().getNewId();%0A    Map%3CString, FileDiffOutput%3E diffList =%0A        oldId == null%0A            ? diffOperations.listModifiedFilesAgainstParent(project, newId, /* parentNum= */ 0)%0A            : diffOperations.listModifiedFiles(project, oldId, newId
 );%0A 
@@ -1932,21 +1932,20 @@
 Summary(
-patch
+diff
 List);%0A 
@@ -1988,27 +1988,45 @@
 ary(
-PatchList patchList
+Map%3CString, FileDiffOutput%3E fileDiffs
 ) %7B%0A
@@ -2066,88 +2066,119 @@
 t%3C%3E(
-patchList.getPatches().size());%0A    for (PatchListEntry e : patchList.getPatches
+fileDiffs.size());%0A    int linesInserted = 0;%0A    int linesDeleted = 0;%0A    for (String path : fileDiffs.keySet
 ()) 
@@ -2203,30 +2203,20 @@
 isMagic(
-e.getNewName()
+path
 )) %7B%0A   
@@ -2248,22 +2248,169 @@
     
-switch (e.getC
+FileDiffOutput fileDiff = fileDiffs.get(path);%0A      linesInserted += fileDiff.insertions();%0A      linesDeleted += fileDiff.deletions();%0A      switch (fileDiff.c
 hang
@@ -2540,39 +2540,148 @@
       r.add(
-e.getNewNam
+%0A              FilePathAdapter.getNewPath(%0A                  fileDiff.oldPath(), fileDiff.newPath(), fileDiff.changeTyp
 e())
+)
 ;%0A          
@@ -2730,54 +2730,218 @@
 add(
-e.getOldName());%0A          r.add(e.getNewName(
+FilePathAdapter.getOldPath(fileDiff.oldPath(), fileDiff.changeType()));%0A          r.add(%0A              FilePathAdapter.getNewPath(%0A                  fileDiff.oldPath(), fileDiff.newPath(), fileDiff.changeType()
 ));%0A
@@ -2998,25 +2998,16 @@
 Summary(
-%0A        
 r.stream
@@ -3045,76 +3045,36 @@
 ew),
-%0A        patchList.getInsertions(),%0A        patchList.getDeletions()
+ linesInserted, linesDeleted
 );%0A 

Switch DiffSummaryLoader to use new diff cache