Skip to content

Inline table don't detect block generated by other macro which contains MetaDataBlock #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Josue-T wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Inline table don't detect block generated by other macro which contains MetaDataBlock #1

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
46f14bc
XLIT-22: Inline table don't detect block generated by other macro whi…
Josue-T Mar 20, 2026
4064ea8
XLIT-22: Inline table don't detect block generated by other macro whi…
Josue-T Apr 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
134 changes: 66 additions & 68 deletions ...e-macro/src/main/java/org/xwiki/contrib/internal/LiveDataInlineTableMacroBlockFilter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@
import org.xwiki.rendering.block.TableCellBlock;
import org.xwiki.rendering.block.TableHeadCellBlock;
import org.xwiki.rendering.block.TableRowBlock;
import org.xwiki.rendering.block.match.ClassBlockMatcher;
import org.xwiki.rendering.renderer.BlockRenderer;
import org.xwiki.rendering.renderer.printer.DefaultWikiPrinter;
import org.xwiki.rendering.renderer.printer.WikiPrinter;
Expand Down Expand Up @@ -413,12 +414,10 @@ private ParsedTable tableToMap(TableBlock table, LiveDataInlineTableMacroParamet
List<TableRowBlock> rows = new ArrayList<>();
int propertiesCount = 0;

for (Block child : table.getChildren()) {
if (child instanceof TableRowBlock) {
TableRowBlock row = (TableRowBlock) child;
rows.add(row);
propertiesCount = Math.max(propertiesCount, row.getChildren().size());
}
for (Block child : table.getBlocks(new ClassBlockMatcher(TableRowBlock.class), Block.Axes.DESCENDANT)) {
Comment thread
Josue-T marked this conversation as resolved.
Outdated
TableRowBlock row = (TableRowBlock) child;
rows.add(row);
propertiesCount = Math.max(propertiesCount, row.getChildren().size());
}
logger.debug("Detected " + propertiesCount + " rows.");

Expand All @@ -440,80 +439,79 @@ private ParsedTable tableToMap(TableBlock table, LiveDataInlineTableMacroParamet
for (TableRowBlock row : rows) {
Map<String, Object> entry = new HashMap<>();
int i = 0;
for (Block child : row.getChildren()) {
if (child instanceof TableCellBlock) {
logger.debug("Parsing a cell of column: " + i);
TableCellBlock cell = (TableCellBlock) child;
WikiPrinter textPrinter = new DefaultWikiPrinter();
for (Block child : row.getBlocks(new ClassBlockMatcher(TableCellBlock.class), Block.Axes.DESCENDANT)) {
Comment thread
Josue-T marked this conversation as resolved.
Outdated
logger.debug("Parsing a cell of column: " + i);
TableCellBlock cell = (TableCellBlock) child;
WikiPrinter textPrinter = new DefaultWikiPrinter();

logger.debug("Rendering cell as text.");
plainTextRenderer.render(cell, textPrinter);

logger.debug("Rendered cell as text: " + textPrinter.toString());
if (entries.isEmpty() && child instanceof TableHeadCellBlock) {
properties.set(i, textPrinter.toString());
inlineHeading = true;
logger.debug("Detected inline heading: " + textPrinter.toString());
}

logger.debug("Rendering cell as text.");
plainTextRenderer.render(cell, textPrinter);
// We need to render the content of the cell as a string so that we can pass it to LiveData.
WikiPrinter cellPrinter = new DefaultWikiPrinter();

logger.debug("Rendered cell as text: " + textPrinter.toString());
if (entries.isEmpty() && child instanceof TableHeadCellBlock) {
properties.set(i, textPrinter.toString());
inlineHeading = true;
logger.debug("Detected inline heading: " + textPrinter.toString());
}
// We need to run transformations in case there is another livedata-inline-table call inside the
// cell.

// We need to render the content of the cell as a string so that we can pass it to LiveData.
WikiPrinter cellPrinter = new DefaultWikiPrinter();
// We have to fiddle with the cell's attributes...
Map<String, String> parsedParameters = new HashMap<>(cell.getParameters());

// We need to run transformations in case there is another livedata-inline-table call inside the
// cell.

// We have to fiddle with the cell's attributes...
Map<String, String> parsedParameters = new HashMap<>(cell.getParameters());
// Update class parameter with our own class.
String classAttr = parsedParameters.getOrDefault(CLASS, "");
parsedParameters.put(CLASS, LIT_CELL_CLASS + " " + classAttr);

Block cellGroup = new GroupBlock(cell.getChildren(), parsedParameters);
logger.debug("Running cell transformations.");
try {
transformationManager.performTransformations(cellGroup,
this.context.getTransformationContext());
} catch (TransformationException e) {
throw new LiveDataInlineTableMacroRuntimeException("Failed to transform cell content.", e);
}

// Update class parameter with our own class.
String classAttr = parsedParameters.getOrDefault(CLASS, "");
parsedParameters.put(CLASS, LIT_CELL_CLASS + " " + classAttr);
logger.debug("Rendering cell as html.");
richTextRenderer.render(cellGroup, cellPrinter);
logger.debug("Rendered cell as html: " + cellPrinter.toString());
entry.put("" + i, cellPrinter.toString());
entry.put("text." + i, textPrinter.toString());
if (fieldsTypes.get(i).equals(DATE)) {
logger.debug("A date is expected, trying to parse.");
String datetimeString = "";
Object timestamp = "";
if (!textPrinter.toString().isBlank()) {

Block cellGroup = new GroupBlock(cell.getChildren(), parsedParameters);
logger.debug("Running cell transformations.");
try {
transformationManager.performTransformations(cellGroup,
this.context.getTransformationContext());
} catch (TransformationException e) {
throw new LiveDataInlineTableMacroRuntimeException("Failed to transform cell content.", e);
}
for (int j = 0; j < this.dateFormats.length; j++) {
try {
logger.debug("Trying to parse date using format: " + this.dateFormats[j]);

Locale locale = this.contextProvider.get().getLocale();
SimpleDateFormat parser = new SimpleDateFormat(this.dateFormats[j], locale);
parser.setLenient(true);

logger.debug("Rendering cell as html.");
richTextRenderer.render(cellGroup, cellPrinter);
logger.debug("Rendered cell as html: " + cellPrinter.toString());
entry.put("" + i, cellPrinter.toString());
entry.put("text." + i, textPrinter.toString());
if (fieldsTypes.get(i).equals(DATE)) {
logger.debug("A date is expected, trying to parse.");
String datetimeString = "";
Object timestamp = "";
if (!textPrinter.toString().isBlank()) {

for (int j = 0; j < this.dateFormats.length; j++) {
try {
logger.debug("Trying to parse date using format: " + this.dateFormats[j]);

Locale locale = this.contextProvider.get().getLocale();
SimpleDateFormat parser = new SimpleDateFormat(this.dateFormats[j], locale);
parser.setLenient(true);

Date date = parser.parse(textPrinter.toString());
datetimeString = new SimpleDateFormat(this.dateFormats[j], locale).format(date);
timestamp = date.toInstant().getEpochSecond();
logger.debug("Parsed date: " + datetimeString);
logger.debug("Parsed unix timestamp: " + timestamp);
entry.put("date." + i, timestamp);
break;
} catch (ParseException e) {
logger.debug("Failed to parse '" + textPrinter.toString() + "' using format "
+ this.dateFormats[j], e);
}
Date date = parser.parse(textPrinter.toString());
datetimeString = new SimpleDateFormat(this.dateFormats[j], locale).format(date);
timestamp = date.toInstant().getEpochSecond();
logger.debug("Parsed date: " + datetimeString);
logger.debug("Parsed unix timestamp: " + timestamp);
entry.put("date." + i, timestamp);
break;
} catch (ParseException e) {
logger.debug("Failed to parse '" + textPrinter.toString() + "' using format "
+ this.dateFormats[j], e);
}
}
}
i++;
}
i++;

}
entries.add(entry);
}
Expand All @@ -533,7 +531,7 @@ private void identifyPropertiesTypes(List<TableRowBlock> rows, List<String> fiel
logger.debug("Detecting the types of columns.");
for (TableRowBlock row : rows) {
int i = 0;
for (Block child : row.getChildren()) {
for (Block child : row.getBlocks(new ClassBlockMatcher(TableCellBlock.class), Block.Axes.DESCENDANT)) {
Comment thread
Josue-T marked this conversation as resolved.
Outdated
if (child instanceof TableCellBlock && !(child instanceof TableHeadCellBlock)) {
Comment thread
Josue-T marked this conversation as resolved.
Outdated
TableCellBlock cell = (TableCellBlock) child;
WikiPrinter textPrinter = new DefaultWikiPrinter();
Expand Down