? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
18 Jul 2019

These files are not used in j4 and were only kept in j3 for potential b/c issues with 3pd admin templates so they can be safely deleted now.

Better to delete them than to continually keep updating them when they're not being used

avatar brianteeman brianteeman - open - 18 Jul 2019
avatar brianteeman brianteeman - change - 18 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jul 2019
Category Administration com_categories com_contact com_content com_fields com_menus com_newsfeeds com_plugins com_tags com_templates
avatar infograf768 infograf768 - change - 19 Jul 2019
Labels Added: ?
avatar brianteeman
brianteeman - comment - 21 Jul 2019

It would be great if this could be tested and merged soon. It makes life so much easier if we're not looking at or trying to fix unused views

avatar alikon
alikon - comment - 21 Jul 2019

I have tested this item successfully on 27b1f1c

Applied pr and played a bit on backed without noticing fault for missing files


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25635.

avatar alikon alikon - test_item - 21 Jul 2019 - Tested successfully
avatar Hackwar
Hackwar - comment - 24 Jul 2019

Please also remove

  • administrator/components/com_categories/tmpl/category/edit_metadata.php
  • administrator/components/com_contact/tmpl/contact/edit_metadata.php
  • administrator/components/com_content/tmpl/article/edit_metadata.php

Please restore

  • administrator/components/com_newsfeeds/tmpl/newsfeed/edit.php
  • administrator/components/com_newsfeeds/tmpl/newsfeed/edit_associations.php
  • administrator/components/com_newsfeeds/tmpl/newsfeed/edit_display.php
  • administrator/components/com_newsfeeds/tmpl/newsfeed/modal.php
avatar brianteeman
brianteeman - comment - 24 Jul 2019

@Hackwar thanks for testing. There is another PR which iirc is for the contact files but I will double check - I will also double check the newsfeeds files

ffaff7b 24 Jul 2019 avatar brianteeman oops
avatar brianteeman
brianteeman - comment - 24 Jul 2019

@Hackwar should be all ok now - thanks

avatar Hackwar
Hackwar - comment - 24 Jul 2019

Looking at the associations file, that is pretty much useless. That one line can very well be moved to the main file and all that boilerplate code could be removed. Haven't looked at the other association files, but I fear they are all the same. In that case that is a bunch of files that could go as well, but maybe in a different PR.

avatar brianteeman
brianteeman - comment - 24 Jul 2019

agree but in a different pr

avatar brianteeman
brianteeman - comment - 24 Jul 2019

This PR is for unused files - another PR for changed files. Single purpose PR :)

avatar wilsonge wilsonge - change - 28 Jul 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-28 00:30:31
Closed_By wilsonge
avatar wilsonge wilsonge - close - 28 Jul 2019
avatar wilsonge wilsonge - merge - 28 Jul 2019
avatar wilsonge
wilsonge - comment - 28 Jul 2019

Thanks!

avatar brianteeman
brianteeman - comment - 28 Jul 2019

Thanks

Add a Comment

Login with GitHub to post a comment