Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pillarsdotnet’s picture

Title: Remove cruft from book_export_html() » Remove assignment to unused variable $export_data from book_export_html() function.
Status: Active » Needs review
FileSize
916 bytes
916 bytes
916 bytes

Patch.

pillarsdotnet’s picture

brianV’s picture

Rolled to HEAD with no changes. Trivial patch, will RTBC once tests pass.

Status: Needs review » Needs work
Issue tags: -Needs backport to D6, -Needs backport to D7

The last submitted patch, book_export_html-remove_unused_variable-1215404-rev2.patch, failed testing.

brianV’s picture

Status: Needs work » Needs review
Issue tags: +Needs backport to D6, +Needs backport to D7
swentel’s picture

Status: Needs review » Reviewed & tested by the community

Still good to go.

catch’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Love these. Committed/pushed to 8.x.

brianV’s picture

Status: Patch (to be ported) » Needs review
FileSize
462 bytes

D7 patch

swentel’s picture

Status: Needs review » Reviewed & tested by the community

Good to go.

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

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