/*
 * Licensed to the Apache Software Foundation (ASF) under one or more contributor license
 * agreements. See the NOTICE file distributed with this work for additional information regarding
 * copyright ownership. The ASF licenses this file to You 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 org.apache.geode.management.internal.cli.commands;

import static org.apache.geode.management.internal.cli.result.ResultBuilder.buildResult;

import java.util.List;
import java.util.Set;

import org.springframework.shell.core.annotation.CliCommand;
import org.springframework.shell.core.annotation.CliOption;

import org.apache.geode.cache.configuration.JndiBindingsType;
import org.apache.geode.distributed.DistributedMember;
import org.apache.geode.distributed.internal.InternalClusterConfigurationService;
import org.apache.geode.management.cli.CliMetaData;
import org.apache.geode.management.cli.Result;
import org.apache.geode.management.internal.cli.exceptions.EntityNotFoundException;
import org.apache.geode.management.internal.cli.functions.CliFunctionResult;
import org.apache.geode.management.internal.cli.functions.DestroyJndiBindingFunction;
import org.apache.geode.management.internal.cli.i18n.CliStrings;
import org.apache.geode.management.internal.cli.result.ResultBuilder;
import org.apache.geode.management.internal.security.ResourceOperation;
import org.apache.geode.security.ResourcePermission;

public class DestroyJndiBindingCommand extends InternalGfshCommand {
  static final String DESTROY_JNDIBINDING = "destroy jndi-binding";
  static final String DESTROY_JNDIBINDING__HELP =
      "Destroy a jndi binding that holds the configuration for the XA datasource.";
  static final String JNDI_NAME = "name";
  static final String JNDI_NAME__HELP = "Name of the binding to be destroyed.";
  static final String IFEXISTS_HELP =
      "Skip the destroy operation when a jndi binding with the same name does not exists. Without specifying this option, this command execution results into an error.";

  @CliCommand(value = DESTROY_JNDIBINDING, help = DESTROY_JNDIBINDING__HELP)
  @CliMetaData(relatedTopic = CliStrings.TOPIC_GEODE_REGION)
  @ResourceOperation(resource = ResourcePermission.Resource.CLUSTER,
      operation = ResourcePermission.Operation.MANAGE)
  public Result destroyJDNIBinding(
      @CliOption(key = JNDI_NAME, mandatory = true, help = JNDI_NAME__HELP) String jndiName,
      @CliOption(key = CliStrings.IFEXISTS, help = IFEXISTS_HELP, specifiedDefaultValue = "true",
          unspecifiedDefaultValue = "false") boolean ifExists) {

    Result result;
    boolean persisted = false;
    InternalClusterConfigurationService service =
        (InternalClusterConfigurationService) getConfigurationService();
    if (service != null) {
      service.updateCacheConfig("cluster", cc -> {
        List<JndiBindingsType.JndiBinding> bindings = cc.getJndiBindings();
        JndiBindingsType.JndiBinding binding = service.findIdentifiable(bindings, jndiName);
        if (binding == null) {
          throw new EntityNotFoundException(
              CliStrings.format("Jndi binding with jndi-name \"{0}\" does not exist.", jndiName),
              ifExists);
        }
        bindings.remove(binding);
        return cc;
      });
      persisted = true;
    }

    Set<DistributedMember> targetMembers = findMembers(null, null);
    if (targetMembers.size() > 0) {
      List<CliFunctionResult> jndiCreationResult =
          executeAndGetFunctionResult(new DestroyJndiBindingFunction(), jndiName, targetMembers);
      return buildResult(jndiCreationResult);
    } else {
      if (persisted) {
        result = ResultBuilder.createInfoResult(CliStrings.format(
            "No members found. Jndi-binding \"{0}\" is removed from cluster configuration.",
            jndiName));
      } else {
        result = ResultBuilder.createInfoResult("No members found.");
      }
    }
    result.setCommandPersisted(persisted);
    return result;
  }
}

@@ -2111,20 +2111,20 @@
 stroy a 
-jndi
+JNDI
  binding
@@ -2157,19 +2157,18 @@
 ion for 
-the
+an
  XA data
@@ -2380,47 +2380,55 @@
 hen 
-a jndi binding with the same name does 
+the specified JNDI binding does %22%0A          + %22
 not 
@@ -2432,17 +2432,16 @@
 ot exist
-s
 . Withou
@@ -2446,19 +2446,8 @@
 out 
-specifying 
 this
@@ -2459,30 +2459,16 @@
 on, 
-this comm
 an
-d
  e
-xecution
+rror
  res
@@ -2476,21 +2476,83 @@
 lts 
-into an error
+from the specification %22%0A          + %22of a JNDI binding that does not exist
 .%22;%0A

Improve gfsh destroy jndi-binding help prose (#1775)