Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mr.infinity created an issue. See original summary.

mr_infinity’s picture

mr_infinity’s picture

Status: Active » Needs review
tfranz’s picture

I received the following error after applying the patch:

Notice: Undefined index: node in file_entity_usage_page() (Zeile 284 von /sites/all/modules/file_entity/file_entity.pages.inc).
Warning: array_keys() expects parameter 1 to be array, null given in file_entity_usage_page() (Zeile 284 von /sites/all/modules/file_entity/file_entity.pages.inc).
Warning: implode(): Argument must be an array in file_entity_usage_page() (Zeile 284 von /sites/all/modules/file_entity/file_entity.pages.inc).

I added a check for $nid not empty in line 283:

           if (!empty($nid)) {
            $nid = implode(array_keys($nid['node']));
           }

… it seems to work well …

michel.settembrino’s picture

I'm using version 7.x-2.21 and this patch works for me too but needs some refactoring.
Updated a newer version.

Status: Needs review » Needs work

The last submitted patch, 5: file-usage-paragraphs-2821914-5.patch, failed testing. View results

michel.settembrino’s picture

Status: Needs work » Needs review
joseph.olstad’s picture

Status: Needs review » Reviewed & tested by the community

looks good

joseph.olstad’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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