Attached is a patch to upgrade the code to work in 6.x. Also fixed:

1) General/debug with firep was not working. This will need to be fixed for 5.x as well. Very easy to fix.
2) Use url() instead of $base_path. This is because url() will handle clean urls appropriately. This will need to be fixed for 5.x as well. Fairly easy to fix.

CommentFileSizeAuthor
drupalforfirebug_6_update.patch3.75 KBjeffcd

Comments

populist’s picture

populist commented
Status: Needs review » Patch (to be ported)

I backported the fixed for firep() and URL to the 5.x branch.

populist’s picture

populist commented
Assigned: Unassigned » populist
waddles’s picture

waddles commented

@populist, can we get a 6.x release please?

populist’s picture

populist commented

There are some memory issues with the way the development snapshot of Firebug for Drupal is working and those need to be resolved first. I hope to resolve those + push out a new release before Drupalcon.

populist’s picture

populist commented
Status: Patch (to be ported) » Fixed

This patch has been applied to the 6.x branch and has been committed. A development snapshot will be coming out soon and I will make some other changes to roll out a Drupal 6 release.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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