J3 Issue ?
avatar mbabker
mbabker
22 Nov 2016

https://github.com/joomla/joomla-cms/blob/staging/modules/mod_finder/mod_finder.php#L25-L39

OpenSearch support in the Smart Search module isn't fully implemented. As seen in the code, the entire functionality is commented out. I remember Rouven purposefully commenting this out in the old Finder Integration repo (deleted long ago), but I don't remember the reason why. This should either be fully implemented (the corresponding view class is in the component, we did add it trying to model from "legacy" Search's implementation) or removed in full.

avatar mbabker mbabker - open - 22 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - change - 22 Nov 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 25 Nov 2016
Category com_finder Modules
avatar brianteeman brianteeman - change - 25 Nov 2016
Status New Confirmed
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 Feb 2017

we did add it trying to model (...) or removed in full.

+1

avatar rvbgnu
rvbgnu - comment - 3 Jun 2017

@mbabker what is the decision about this?

avatar mbabker
mbabker - comment - 3 Jun 2017

One hasn't been made yet. I wish I could remember the reason it was disabled, but it seems like we should be looking to get it enabled.

avatar franz-wohlkoenig franz-wohlkoenig - change - 8 Nov 2017
Status Confirmed Discussion
avatar brianteeman brianteeman - labeled - 25 Mar 2018
avatar Hackwar
Hackwar - comment - 21 May 2018

Can you assign this issue to me, so that I can tackle this on my list of finder issues, which is getting longer and longer...? ?

avatar brianteeman
brianteeman - comment - 21 May 2018

@Hackwar we can only assign people who are part of the maintainers team - limitation of github :(

avatar Hackwar
Hackwar - comment - 29 Jun 2018

This is now fixed in #20936

avatar brianteeman brianteeman - change - 29 Jun 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-06-29 23:57:57
Closed_By brianteeman
Labels Added: J3 Issue
avatar brianteeman brianteeman - close - 29 Jun 2018

Add a Comment

Login with GitHub to post a comment