The current file usage tracking for blocks only applies to the block_admin_configure form. It should also apply to the block_add_block_form form.

A check to ensure that the block body field exists before attempting to inspect it for file IDs should also be added. Only custom blocks have a body field.

CommentFileSizeAuthor
#2 3440404-2.patch8.71 KBdevin carlson
#2 3440404-2-tests-only.patch4.88 KBdevin carlson

Comments

Devin Carlson created an issue. See original summary.

devin carlson’s picture

devin carlson commented
Status: Active » Needs review
StatusFileSize
new 3440404-2-tests-only.patch4.88 KB
new 3440404-2.patch8.71 KB

A patch to add file usage tracking to the block_add_block_form form and add a check for the body field before attempting to use it.

The last submitted patch, 2: 3440404-2-tests-only.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

devin carlson’s picture

devin carlson commented
Status: Needs review » Fixed

Tested #2 and committed to Editor 7.x-1.x.

  • Devin Carlson committed 81c633f7 on 7.x-1.x 
    Issue #3440404: File usage tracking is missing on block add form

Status: Fixed » Closed (fixed)

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