// Copyright (C) 2017 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.change;

import com.google.gerrit.extensions.restapi.Response;
import com.google.gerrit.extensions.restapi.RestApiException;
import com.google.gerrit.extensions.restapi.RestModifyView;
import com.google.gerrit.extensions.webui.UiAction;
import com.google.gerrit.server.StarredChangesUtil;
import com.google.gwtorm.server.OrmException;
import com.google.inject.Inject;
import com.google.inject.Singleton;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

@Singleton
public class Unignore
    implements RestModifyView<ChangeResource, Unignore.Input>, UiAction<ChangeResource> {
  private static final Logger log = LoggerFactory.getLogger(Unignore.class);

  public static class Input {}

  private final StarredChangesUtil stars;

  @Inject
  Unignore(StarredChangesUtil stars) {
    this.stars = stars;
  }

  @Override
  public Description getDescription(ChangeResource rsrc) {
    return new UiAction.Description()
        .setLabel("Unignore")
        .setTitle("Unignore the change")
        .setVisible(canUnignore(rsrc));
  }

  @Override
  public Response<String> apply(ChangeResource rsrc, Input input)
      throws RestApiException, OrmException {
    // Don't try to unignore own changes or not ignored changes
    if (canUnignore(rsrc)) {
      stars.unignore(rsrc);
    }
    return Response.ok("");
  }

  private boolean canUnignore(ChangeResource rsrc) {
    return !rsrc.isUserOwner() && isIgnored(rsrc);
  }

  private boolean isIgnored(ChangeResource rsrc) {
    try {
      return stars.isIgnored(rsrc);
    } catch (OrmException e) {
      log.error("failed to check ignored star", e);
    }
    return false;
  }
}

@@ -1651,27 +1651,25 @@
 Visible(
-canUni
+isI
 gnore
+d
 (rsrc));
@@ -1806,87 +1806,21 @@
     
-// Don't try to unignore own changes or not ignored changes%0A    if (canUni
+if (isI
 gnore
+d
 (rsr
@@ -1896,117 +1896,8 @@
  %7D%0A%0A
-  private boolean canUnignore(ChangeResource rsrc) %7B%0A    return !rsrc.isUserOwner() && isIgnored(rsrc);%0A  %7D%0A%0A
   pr

Allow to unignore own changes