Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#18761] - [4.0] Remove dummy JSite class

? J4 Issue ?
avatar wilsonge
wilsonge
20 Nov 2017

Steps to reproduce the issue

Given we've totally removed the JSite alias - either this class is going to fail super hard or isn't being used or there's some hella voodoo going on. https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_search/helpers/site.php

Let's figure out a way to get rid of it :)

avatar wilsonge wilsonge - open - 20 Nov 2017
avatar joomla-cms-bot joomla-cms-bot - change - 20 Nov 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 20 Nov 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Nov 2017
Category ⇒ Code style
avatar brianteeman brianteeman - change - 25 Mar 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 25 Mar 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 7 Apr 2018
Status New ⇒ Discussion
avatar brianteeman
brianteeman - comment - 3 Aug 2018

anyone??

avatar wojsmol
wojsmol - comment - 3 Aug 2018

@wilsonge If github search is up-to-date then we only use this class once in com_search model.

cc: @Hackwar

avatar franz-wohlkoenig franz-wohlkoenig - change - 7 Apr 2019
Labels Added: ?
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 7 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 9 Apr 2019
Category Code style ⇒
avatar wilsonge wilsonge - change - 5 May 2019
Status Discussion ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2019-05-05 16:18:17
Closed_By ⇒ wilsonge
avatar wilsonge
wilsonge - comment - 5 May 2019

This has been done

avatar wilsonge wilsonge - close - 5 May 2019

Add a Comment

Login with GitHub to post a comment