/*
 *  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.isis.viewer.wicket.ui.util;

import org.apache.wicket.Component;
import org.apache.wicket.MarkupContainer;
import org.apache.wicket.markup.html.basic.Label;

import org.apache.isis.viewer.wicket.ui.ComponentType;

public final class Components {

    private Components() {
    }

    /**
     * Permanently hides by replacing with a {@link Label} that has an empty
     * string for its caption.
     */
    public static void permanentlyHide(final MarkupContainer container, final String... ids) {
        for (final String id : ids) {
            permanentlyHideSingle(container, id);
        }
    }

    /**
     * @see #permanentlyHide(MarkupContainer, String...)
     */
    public static void permanentlyHide(final MarkupContainer container, final ComponentType... componentIds) {
        for (final ComponentType componentType : componentIds) {
            final String wicketId = componentType.getWicketId();
            permanentlyHideSingle(container, wicketId);
        }
    }

    /**
     * Not overloaded because - although compiles ok on JDK6u20 (Mac), fails to
     * on JDK6u18 (Ubuntu)
     */
    private static void permanentlyHideSingle(final MarkupContainer container, final String id) {
        final Label label = new Label(id, "");
        label.setVisible(false);
        container.addOrReplace(label);
    }

    /**
     * Sets the visibility of the child component(s) within the supplied
     * container.
     */
    public static void setVisible(final MarkupContainer container, final boolean visibility, final String... ids) {
        for (final String id : ids) {
            setVisible(container, visibility, id);
        }
    }

    /**
     * @see #setVisible(MarkupContainer, boolean, String...)
     */
    public static void setVisible(final MarkupContainer container, final boolean visibility, final ComponentType... componentTypes) {
        for (final ComponentType componentType : componentTypes) {
            final String wicketId = componentType.getWicketId();
            setVisible(container, visibility, wicketId);
        }
    }

    private static void setVisible(final MarkupContainer container, final boolean visibility, final String wicketId) {
        final Component childComponent = container.get(wicketId);
        childComponent.setVisible(visibility);
    }

}

@@ -946,16 +946,73 @@
 tainer;%0A
+import org.apache.wicket.markup.html.WebMarkupContainer;%0A
 import o
@@ -2127,38 +2127,64 @@
 nal 
-Label label = new Label(id, %22%22
+WebMarkupContainer invisible = new WebMarkupContainer(id
 );%0A 
@@ -2190,21 +2190,25 @@
         
-label
+invisible
 .setVisi
@@ -2254,13 +2254,17 @@
 ace(
-label
+invisible
 );%0A 

Use WebMarkupContainer instead of Label with empty string