title says it all

CommentFileSizeAuthor
#3 picasa_guser-207105-3.patch861 byteshairymnstr

Comments

cyberswat’s picture

cyberswat commented
Status: Active » Postponed (maintainer needs more info)

I copied the layout from googles existing picasa site ... I'm seeing everything displaying as I intended on the test site you added your albums to ... coudl you post a screenshot of what you are seeing?

cyberswat’s picture

cyberswat commented
Assigned: Unassigned » cyberswat
Status: Postponed (maintainer needs more info) » Closed (fixed)
hairymnstr’s picture

hairymnstr commented
Assigned: cyberswat » hairymnstr
Status: Closed (fixed) » Patch (to be ported)
StatusFileSize
new picasa_guser-207105-3.patch861 bytes

The bug doesn't show up in Firefox, but is apparent in IE7 see below:

http://rozamira.ws/images/drupal/ie7.png

It is also apparent using html Tidy as it throws up a syntax error for every album displayed currently.

Looking at the code, it appears to be some old code that was left after implementing _picasa_guser_get_album_teaser() as a function. Removing the two lines following that call removes all errors with no apparent side effects.

I have attached a patch to picasa_guser.module to be included in the next bug fix release.

cyberswat’s picture

cyberswat commented
Status: Patch (to be ported) » Closed (fixed)

I'm decommissioning the d5 version so closing out this ticket.