Skip to content
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

Jump to bottom

Table format ignores link parameter for values #3545

Open
JeroenDeDauw opened this issue Nov 23, 2018 · 2 comments
Open

Table format ignores link parameter for values #3545

JeroenDeDauw opened this issue Nov 23, 2018 · 2 comments
Labels
bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error

Comments

@JeroenDeDauw
Copy link
Member

JeroenDeDauw commented Nov 23, 2018 •
edited by kghbln

Setup

  • MediaWiki | 1.31.1 (4b70558)22:37, 19. Nov. 2018
  • PHP | 7.0.30-0+deb9u1 (apache2handler)
  • MariaDB | 10.1.37-MariaDB-0+deb9u1
  • Elasticsearch | 6.5.0
  • Semantic MediaWiki | 3.1.0-alpha (c748927) 07:35, 22. Nov. 2018
  • Maps | 5.6 (fa555bf) 15:26, 16. Jul. 2018

Issue

Note link=none:

{{#ask:[[Coordinates::+]]|?Coordinates|?Description|?URL|format=table|link=none}}

image

The property links get removed but the value one is still there

Example

https://sandbox.semantic-mediawiki.org/wiki/Issue/3545
@JeroenDeDauw JeroenDeDauw changed the title Table format ignores link parameter Table format ignores link parameter for values Nov 23, 2018
@JeroenDeDauw JeroenDeDauw added the bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error label Nov 23, 2018
@kghbln
Copy link
Member

kghbln commented Nov 23, 2018 •
edited

the value one

I believe with value of the URL is meant. I think this is a 2.1 regression or even earlier.

@JeroenDeDauw
Copy link
Member Author

I meant: values (as opposed to properties and subjects) are still linked. (That only happens if the values are of a certain type, such as URL or Page.)

@krabina krabina mentioned this issue Oct 7, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Occurrence of an unintended or unanticipated behaviour that causes a vulnerability or fatal error
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JeroenDeDauw @kghbln