? Success

User tests: Successful: Unsuccessful:

avatar smz
smz
18 Dec 2014

Should we expel those two rascals?

avatar smz smz - open - 18 Dec 2014
avatar jissues-bot jissues-bot - change - 18 Dec 2014
Labels Added: ?
avatar Bakual Bakual - close - 18 Dec 2014
avatar Bakual Bakual - change - 18 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-18 19:10:37
avatar shur
shur - comment - 19 Dec 2014

Perhaps we can even remove these 2 files:
/administrator/templates/hathor/html/com_finder/filter/index.html
/administrator/templates/hathor/html/com_finder/statistics/index.html

Since these directories do not contain anything (except index.html), they can be removed too:
/administrator/templates/hathor/html/com_finder/filter/
/administrator/templates/hathor/html/com_finder/statistics/

avatar smz
smz - comment - 19 Dec 2014

@shur I think (but I'm unsure) those two directories are needed in case you create filters and/or gather statistics...

avatar brianteeman
brianteeman - comment - 19 Dec 2014

unlikely as those are template overrides

avatar smz
smz - comment - 19 Dec 2014

@brianteeman correct!
@shur this PR has already been merged: do you want to make a news one?

avatar shur
shur - comment - 19 Dec 2014

@snz since these folders are empty, they are not involved in template overrides. So they can be deleted.

do you want to make a new one?

No, I don't. It's up to you.

avatar smz
smz - comment - 19 Dec 2014

OK!

avatar smz
smz - comment - 19 Dec 2014

Done in #5471

Add a Comment

Login with GitHub to post a comment