Skip to content

Unused att--filename parameter in Layout cell macro #174

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.

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

Closed
petrenkonikita112263 opened this issue Oct 27, 2023 · 2 comments
Closed

Unused att--filename parameter in Layout cell macro #174

petrenkonikita112263 opened this issue Oct 27, 2023 · 2 comments

Comments

@petrenkonikita112263
Copy link

petrenkonikita112263 commented Oct 27, 2023

Prerequisites:

  • XWiki instance 13.10
  • Latest version of Pro Macros
  • Trial license
  • Having at least one attachment (any type) on a test page

Step to reproduce:

  1. Go into Test Page, start editing it via Wiki mode.
  2. Copy/paste this line of code. Don't forget to replace filename with the existing one on your page.
{{layout-cell att--filename="image.jpg"}}
Lorem Ipsum is simply dummy text of the printing and typesetting industry.
{{/layout-cell}}
  1. Save it.

Actual result
Despite of specified attachment, only text in a cell will be visible.
image

Expected result
I see that field att--filename was introduced in commit 5ef0116 in LayoutCell.xml file. I don't know what was the intention of it to display image/player/viewer in a cell, but if it has no effect, it can be removed other macros of Layout don't use it. That's why only content of macro shall be as input for this macro

@petrenkonikita112263
Copy link
Author

petrenkonikita112263 commented Oct 30, 2023

From my last paragraph, instead of looking at this as bug, it can be a slight improvement by removing valueless field "att--filename" with further renaming the title (and label) of this issue.

@raphj raphj changed the title Providing filename of attachment in Layout cell macro has no expected result - only text is visible Unused att--filename parameter in Layout cell macro Nov 13, 2023
@raphj
Copy link
Contributor

raphj commented Nov 13, 2023

I removed the unused parameter. I haven't found a way to use this parameter in Confluence. I suspect it comes from a copy paste that was not completely cleaned up. We can always properly implement the parameter later if it indeed exists. FYI @acotiuga.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants