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.

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

Datatables format / Sort option is not respected in ask queries #211

Closed
djflux opened this issue May 3, 2017 · 4 comments
Closed

Datatables format / Sort option is not respected in ask queries #211

djflux opened this issue May 3, 2017 · 4 comments

Comments

@djflux
Copy link

djflux commented May 3, 2017

Setup

  • MW version: 1.25.5
  • DB (MySQL etc.): MySQL 5.6.36
  • PHP version: 5.6.14 (apache2handler)
  • SMW version: 2.4.0-rc1
  • SRF version: 2.3
  • Browsers and versions (if applicable): NA

Issue

The Datatables results format does not respect the sort option set.

A demonstration is available:

Production SMW

https://www.semantic-mediawiki.org/wiki/User:Arechenberg

Sandbox SMW

https://sandbox.semantic-mediawiki.org/wiki/Utilisateur:Arechenberg

Here is are the ask queries I tried

Production SMW site

modified from  https://www.semantic-mediawiki.org/wiki/Demo:Datatables_simple_table

{{#ask:
 [[Category:Result formats]]
 [[Release status::current]]
 [[Docinfo language::en]]
 |?Has format name=Name
 |?Has description=Description
 |?Has category=Category
 |?Has component=Provided by 	
 |format=datatables
 |mainlabel=Help page
 |sort=Has category
 |order=descending
}}
Sandbox SMW

{{#ask:
 [[-Has date::+]]
 [[-Has number::+]]
 [[Category:Test]]
 |?Page author
 |?Has date
 |?Has number	
 |format=datatables
 |sort=Has number
 |order=descending
}}

Thanks for your help.

Cheers,
Andy

@kghbln
Copy link
Member

kghbln commented May 3, 2017

Thanks for testing and reporting. Indeed this is a missing feature for the "datatables" format. SWM 3.0 will bring the class datatable to the "table" and "broadtable" formats (See this pull request). Result format "datatables" indeed still needs this enhancement.

@mwjames
Copy link
Contributor

mwjames commented May 3, 2017 via email

@kghbln
Copy link
Member

kghbln commented May 3, 2017

I'm aware of that but as I already mentioned in #2427

I knew about it but did not remember where you wrote it. I just cross-referenced for completeness.

@krabina
Copy link
Contributor

krabina commented Mar 24, 2023

Closing since this relates to a very old version. Please try the latest version and reopen should the problem persist.

@krabina krabina closed this as completed Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants