Word wrap on multiple row and column table

[rathnapandi]

I am facing issue with word wrap on tables. If first column is wrapped, second column alignment is gone (starts with next line of first column). Find below the output.

java code

import picocli.CommandLine;

import java.util.ArrayList;
import java.util.List;
import java.util.UUID;

@CommandLine.Command
public class MainTest implements Runnable{

    @CommandLine.Spec
    CommandLine.Model.CommandSpec spec;

    public static void main(String[] args) {
        new CommandLine(MainTest.class).execute(args);
    }

    @Override
    public void run() {
        System.out.println(spec);
        List<String> headerNames = List.of("Cluster Name", "Cluster RefId", "Cluster Type");
        List<Integer> columnWidths = List.of(20, 40, 25);
        CommandLine.Help.ColorScheme colorScheme = new CommandLine.Help.ColorScheme.Builder()
            .commands(CommandLine.Help.Ansi.Style.bold, CommandLine.Help.Ansi.Style.fg_cyan)
            .build();


        spec.usageMessage().autoWidth(true);
        List<CommandLine.Help.Column> columnList = new ArrayList<>(headerNames.size());
        for (int eachColumnWidth : columnWidths) {
            columnList.add(new CommandLine.Help.Column(eachColumnWidth, 0, CommandLine.Help.Column.Overflow.WRAP));
        }
        CommandLine.Help.TextTable textTable = CommandLine.Help.TextTable.forColumns(spec.commandLine().getColorScheme(),  columnList.toArray(CommandLine.Help.Column[]::new));
        textTable.indentWrappedLines = 0;

        CommandLine.Help.Ansi.Text[] headerNamesWithColorScheme = headerNames.stream().map(colorScheme::commandText).toArray(CommandLine.Help.Ansi.Text[]::new);
        textTable.addEmptyRow();
        textTable.addRowValues(headerNamesWithColorScheme);

        textTable.addRowValues("Shared Data Plane", UUID.randomUUID().toString(), "on prem");
        textTable.addRowValues("Shared Data Plane on AWS", UUID.randomUUID().toString(), "cloud");
        System.out.println(textTable);
    }
}

[remkop]

@rathnapandi yes, that is how the TextTable class is currently implemented:
If a cell is either spanning multiple cells or wrapped, the value for the following column is written on the next row.
(I don't remember exactly why I did this, I think this is to deal with some edge cases with very long values.)

One idea to avoid this is to subclass CommandLine.Help.TextTable and override public void addRowValues(Text... values).
You may need to experiment a bit to get it right.
Let me know if your solution is reusable as an improvement to CommandLine.Help.TextTable!

[rathnapandi]

Thanks @remkop for the suggestion.

Just added a dirty fix to overcome the issue.

public void addRowValues(CommandLine.Help.Ansi.Text... values) {
        if (values.length > columns.length) {
            throw new IllegalArgumentException(values.length + " values don't fit in " +
                columns.length + " columns");
        }
        addEmptyRow();
        int oldIndent = unindent(values);
        int wrap = - 1;
        for (int col = 0; col < values.length; col++) {
            int row = rowCount() + wrap;// write to last row: previous value may have wrapped to next row
            putValue(row, col, values[col]);
            CommandLine.Help.Column column = this.columns[col];
            if(column.overflow == CommandLine.Help.Column.Overflow.WRAP)
                wrap = row - rowCount();
        }
        reindent(oldIndent);
    }