Just tried the latest version from CVS, and got following two issues:
1. Seems the ImageBlock couldn't change "Random Photo" to any other, such as "Recent Photo" and "Recent Album", although the display of heading, title, date works perfectly.
2. Don't know if there is a way to configure the thumbnail size or not. I didn't want to change the global thumbnail size, but just want to change the thumnail displaced in Drupal block, is this possible?

Thanks.

Comments

syntaxerror’s picture

syntaxerror CreditAttribution: syntaxerror commented

It doesn't look like I can use anything else either... hrm.

walkah’s picture

walkah CreditAttribution: walkah commented
Assigned: Unassigned » walkah

will look into both issues

kiz_0987’s picture

kiz_0987 CreditAttribution: kiz_0987 commented
Assigned: walkah » Unassigned

There are a couple of bugs in gallery.module.

In v1.8 (line 149) and the current cvs v1.9 (line 160).

Replace 

'gallery_block_block' . $delta,

With

'gallery_block_block_' . $delta,

Replace $params['block'] with $params['blocks'] (2 places)

The size is hardcoded (for now). Look for $params['maxSize'] = 160; and change it.

npe’s picture

npe CreditAttribution: npe commented

Thank Walkah, it works on my server. :)

npe’s picture

npe CreditAttribution: npe commented
Status: Active » Fixed
pitpit’s picture

pitpit CreditAttribution: pitpit commented
Status: Fixed » Active

it seems to have not been fixed on 1.10 !

pitpit’s picture

pitpit CreditAttribution: pitpit commented
Status: Active » Fixed
pitpit’s picture

pitpit CreditAttribution: pitpit commented
Status: Fixed » Needs review
alexandreracine’s picture

alexandreracine CreditAttribution: alexandreracine commented

+ 1 for this patch.

I confirm that "recent album", and "random photo" works with the patch of kiz_0987.

walkah’s picture

walkah CreditAttribution: walkah commented

committed to HEAD. thanks guys.

walkah’s picture

walkah CreditAttribution: walkah commented
Status: Needs review » Fixed
kiz_0987’s picture

kiz_0987 CreditAttribution: kiz_0987 commented
Status: Fixed » Needs review

Walkah - thanks for committing this patch, but you missed a bit. Check comment #3 and you will see that 2 items need changing (you have done one). The other part is to replace $params['block'] with $params['blocks'] (2 places)

darius’s picture

darius CreditAttribution: darius commented
Status: Needs review » Closed (duplicate)

The issue continues on http://drupal.org/node/31412

Darius