Files: 
CommentFileSizeAuthor
#6 2000006-filetestform-file-drupal-6.patch7.7 KBjuampy
PASSED: [[SimpleTest]]: [MySQL] 55,487 pass(es).
[ View ]
#6 interdiff.txt657 bytesjuampy
#2 2000006-filetestform-file-drupal-2.patch7.54 KBACF
PASSED: [[SimpleTest]]: [MySQL] 56,528 pass(es).
[ View ]
#1 2000006-filetestform-file-1.patch7.73 KBACF
PASSED: [[SimpleTest]]: [MySQL] 56,038 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new7.73 KB
PASSED: [[SimpleTest]]: [MySQL] 56,038 pass(es).
[ View ]

Patch attached.

StatusFileSize
new7.54 KB
PASSED: [[SimpleTest]]: [MySQL] 56,528 pass(es).
[ View ]

Oops left in some unnecessary code.

Tagging.

Status:Needs review» Reviewed & tested by the community

Patch applies cleanly.

File uploading tests pass successfully after applying.

I have also had a look at the code of the patch and looks OK to me.

Status:Reviewed & tested by the community» Needs work

+++ b/core/modules/file/tests/file_test/file_test.moduleundefined
@@ -19,9 +19,7 @@
function file_test_menu() {
   $items['file-test/upload'] = array(
     'title' => 'Upload test',
-    'page callback' => 'drupal_get_form',
-    'page arguments' => array('_file_test_form'),
-    'access arguments' => array('access content'),
+    'route_name' => 'file_test',
     'type' => MENU_CALLBACK,
   );
   return $items;

Since this is a MENU_CALLBACK I think we can get rid of the entire file_test_menu function.

Status:Needs work» Needs review
StatusFileSize
new657 bytes
new7.7 KB
PASSED: [[SimpleTest]]: [MySQL] 55,487 pass(es).
[ View ]

Removed file_test_menu().

File upload tests still pass.

Status:Needs review» Reviewed & tested by the community

That's fine now.

Status:Reviewed & tested by the community» Fixed

Committed eb354a5 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.