Needs review
Project:
Facebook Comments Box
Version:
6.x-1.0
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2012 at 14:36 UTC
Updated:
10 Mar 2014 at 06:07 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Bastlynn commentedPatches attached below.
Comment #2
Bastlynn commentedCorrecting a minor issue n the D6 patch.
Comment #3
Bastlynn commentedGah, found another set of minor typo bugs. That's what I get for coding while sick. Corrected patches attached.
Comment #4
Bastlynn commentedAnd one last round.
Comment #5
Bastlynn commentedSetting to needs review.
Comment #6
dandaman commentedI tested the Drupal 6 patch and it worked very well for my site. Thanks for the patch!
Comment #6.0
dandaman commentedFixing code display.
Comment #7
Bastlynn commentedComment #8
Bastlynn commentedComment #9
makangus commentedI don't think we need both isset and !empty on this line
if (isset($app_id) && !empty($app_id)) {if (!empty($app_id)) {should do it.Comment #10
shahidbscs commentedThank you so much Bastlynn, very nice working.
It will be very appreciated if you update solution on D7 release :)