Mailing-List: contact user-help@karaf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: user@karaf.apache.org
Received-SPF: pass (nike.apache.org: domain of cschneider111@gmail.com
 designates 209.85.216.172 as permitted sender)
Sender: Christian Schneider <cschneider111@gmail.com>
Message-ID: <52C6A7E7.7060709@die-schneider.net>
Date: Fri, 03 Jan 2014 13:07:03 +0100
From: Christian Schneider <chris@die-schneider.net>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:24.0) Gecko/20100101 Thunderbird/24.2.0
MIME-Version: 1.0
To: user@karaf.apache.org
Subject: Re: karaf 3.0 : ShellTable question
References: <1388741711572-4030886.post@n3.nabble.com>
In-Reply-To: <1388741711572-4030886.post@n3.nabble.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Hi Luca,

I think my original idea was that all methods with just a subject as 
name were thought to return Shelltable so they could be chained.
As this is already not the case for column it is not consistent though.

So I think it makes sense to add a new method Row row() and also Row 
row(Object... content).

Should we deprecated the old addRow method to avoid duplication and show 
people how to use the interface?
Btw. I would also like to adapt the tests to reflect the main way to use 
Shelltable.

Christian


Am 03.01.2014 10:35, schrieb lbu:
> Hi,
> I've started playing with Karaf 3.0 (thanks for the Christmas gift) and I
> have some questions related to ShellTable:
>
> For most of the methods, the action is not in the name so you have
> column(...),  separator(...), etc but this is not true for addRow:
>      
>      public Row addRow() {
>          Row row = new Row();
>          rows.add(row);
>          return row;
>      }
>
> Is there any reason?
>
> For my own needs, I've also created an EnhancedShellTable which let you to:
> - set-up columns in the constructor
>      
>      public EnhancedShellTable(String... columns) {
>          super();
>          for(String column : columns) {
>              super.column(column);
>          }
>      }
>
> - add and fill a row with a single method
>
>      public void row(Object... content) {
>          addRow().addContent(content);
>      }
>
> Would it be possible to add them to ShellTable?
>
> Regards,
> Luca
>
>
>
>
>
> --
> View this message in context: http://karaf.922171.n3.nabble.com/karaf-3-0-ShellTable-question-tp4030886.html
> Sent from the Karaf - User mailing list archive at Nabble.com.


-- 
  
Christian Schneider
http://www.liquid-reality.de

Open Source Architect
Talend Application Integration Division http://www.talend.com