// Copyright (C) 2009 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.sshd.commands;

import com.google.common.cache.Cache;
import com.google.common.collect.Sets;
import com.google.gerrit.extensions.registration.DynamicMap;
import com.google.gerrit.sshd.SshCommand;
import com.google.inject.Inject;

import java.util.SortedSet;

abstract class CacheCommand extends SshCommand {
  @Inject
  protected DynamicMap<Cache<?, ?>> cacheMap;

  protected SortedSet<String> cacheNames() {
    SortedSet<String> names = Sets.newTreeSet();
    for (DynamicMap.Entry<Cache<?, ?>> e : cacheMap) {
      names.add(cacheNameOf(e.getPluginName(), e.getExportName()));
    }
    return names;
  }

  protected String cacheNameOf(String plugin, String name) {
    if ("gerrit".equals(plugin)) {
      return name;
    } else {
      return plugin + "." + name;
    }
  }
}

// Copyright (C) 2009 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.sshd.commands;

import com.google.common.cache.Cache;
import com.google.common.collect.Sets;
import com.google.gerrit.extensions.registration.DynamicMap;
import com.google.gerrit.sshd.SshCommand;
import com.google.inject.Inject;

import java.util.SortedSet;

abstract class CacheCommand extends SshCommand {
  @Inject
  protected DynamicMap<Cache<?, ?>> cacheMap;

  protected SortedSet<String> cacheNames() {
    SortedSet<String> names = Sets.newTreeSet();
    for (DynamicMap.Entry<Cache<?, ?>> e : cacheMap) {
      names.add(cacheNameOf(e.getPluginName(), e.getExportName()));
    }
    return names;
  }

  protected String cacheNameOf(String plugin, String name) {
    if ("gerrit".equals(plugin)) {
      return name;
    } else {
      return plugin + "-" + name;
    }
  }
}

Use '-' in cache name as separator between plugin name and cache name
Use '-' in cache name as separator between plugin name and cache name

In other places we already use '-' as separator in names that include
the plugin name, e.g. for capabilities that are defined by a plugin.
To be consistent the cache name should use the same separator. '.'
cannot be used for capabilities since it is not allowed in parameter
names in Git config files.

Change-Id: Ic1a669930831a1d4e526f2a22771879ecf7f19c1
Signed-off-by: Edwin Kempin <b444e279ad95fdef4fbdd82c813b624595df204a@sap.com>
