Download & Extend

nodeorder.install success message issue

Project:nodeorder
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)
Issue tags:installation

Issue Summary

Installation/uninstallation of nodeorder reports failure, regardless of actual success or failure.

Problem is that drupal_add_field() and drupal_drop_field() return nested arrays, while the installation functions expect a single flat array.

ie: instead of

if ($ret['success'] === TRUE) { ... }

it should go something like

if ($ret[$i]['success'] === TRUE) { ... }

where $i is an integer. The attached patch iterates through the nested arrays and checks for error values.

AttachmentSize
nodeorder.install.fixed_success_checks.patch2.53 KB

Comments

#1

Status:needs review» reviewed & tested by the community

Patch commited and will show up in the next new dev release.

#2

Status:reviewed & tested by the community» fixed

#3

Status:fixed» closed (fixed)

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

nobody click here