461421 - Avatar images and supporting code

Status: RESOLVED FIXED

(Mozilla Messaging Graveyard :: Server Operations, defect)

Description

[Paul Booker]

r19314

would you please check out on stage , thanks

Comment 1

[Philippe M. Chiasson (:gozer)]

Done. Needs to be verified.

Comment 2

[Paul Booker]

I can't see the avatars on stage at the moment but i can see them on my local server running the most recent database from the stage server.

When i run "svn diff" under the directory stb on my local machine the only change to push is a correction to show the avatar against a comment.

Not sure what the problem is at the moment i can see the avatars are on  stage by navigating to them directly and can see that the modification to stb/template.php to remove the footer is working so everything looks to have been checked out to stage  .

Investigating ...

-Paul

Comment 3

[Paul Booker]

Attachment: Showing avatars rendered on my local server

Comment 4

[Philippe M. Chiasson (:gozer)]

Looks to me, from brief investigation that, in template.php, 
stb_preprocess_page() gets invoked, but acquia_marina_preprocess_node() doesn't get called at all

Comment 5

[Paul Booker]

Attachment: Developer module shows that our derived theme is not being pulled on stage server - 1

Comment 6

[Paul Booker]

Attachment: Developer module shows that our derived theme is not being pulled on stage server - 2

Investigating ..

Comment 7

[Philippe M. Chiasson (:gozer)]

since the stb_preprocess_page() gets invoked oaky, shouldn't the other one maybe be called stb_preprocess_node() ?

Comment 8

[Paul Booker]

The contributed theme acquia is under /themes instead of /sites/all/themes.

Normally /themes is reserved for core themes and /sites/all/themes is reserved
for contributed themes . Would you place the acquia theme and its stb
derivative under /sites/all/themes

Thanks Paul

Comment 9

[Philippe M. Chiasson (:gozer)]

I've moved it around, and except from temporarly breaking the site, it's still not showing up as you'd like, and the acquia_marina_preprocess_node() function isn't being called.

Comment 10

[Paul Booker]

Were still using the acquia_marina theme under /themes instead of sites/all/themes

Comment 11

[Philippe M. Chiasson (:gozer)]

Yes, but this time around they are both installed under /themes, shouldn't that be good enough ?

Comment 12

[Paul Booker]

No, can we have the theme under /sites/all/themes . Its best to stick to the plan then everything should work out (fingers crossed)

The problem looks to at least partly reside with $avatars_dir which looks for the avatars only in the theme directory sites/all/themes/acquia_marina/stb/images


function stb_avatars($uid, $name) {  

    $avatars_dir = 'sites/all/themes/acquia_marina/stb/images/avatars/';
    //some users have personal avatars
    $avatar = $avatars_dir ."_{$uid}.gif";

Comment 13

[Philippe M. Chiasson (:gozer)]

Definitely not the problem, as acquia_marina_preprocess_node and acquia_marina_preprocess_comment never even get called in the first place.

Comment 14

[Philippe M. Chiasson (:gozer)]

Just FYI, I've moved them all under sites/all/themes/ and it behaves exactly the same way.

Comment 15

[Paul Booker]

I can't see what the problem is at the moment .

On my local server i have the same contributed modules as on stage the same
code that is in SVN (which i'm sure is checked out on stage) and the same
database that's on stage but on my local server the avatars are all rendered
fine.

I'll keep on searching for the solution ...

Thanks for all you help Gozer .
Paul

Comment 16

[jamey]

paul here 

http://stage.spreadthunderbird.com/sites/all/themes/acquia_marina/stb/images/avatars/_10.gif

Comment 17

[Philippe M. Chiasson (:gozer)]

What version of drupal are you running? 6.4, 6.5 or 6.6 ?

Comment 18

[Paul Booker]

6.5

Comment 19

[Philippe M. Chiasson (:gozer)]

stage is running 6.4 with the 2 security fixes of 6.5 and 6.6 applied. This could be the problem. I am going to try and upgrade to 6.5, see what happens.

Comment 20

[Paul Booker]

thanks ,

if that doesn't work would you mind clarifying whats under /sites/all/themes/acquia_marina/stb

Comment 21

[Philippe M. Chiasson (:gozer)]

Attachment: All files under sites/all

Here it is, the list of all files under drupal/sites/all/

Comment 22

[Paul Booker]

r19345

Committed the rest of my  work . 

Would you update stage, thanks.


Has the acquia theme from under /themes been removed, just wandering if there is some kind of clash causing the problem.

Paul

Comment 23

[Philippe M. Chiasson (:gozer)]

Merged that revision to tags/staging and updated staging server.

I moved the theme away, so it can't be clashing.

Comment 24

[Paul Booker]

Two more ideas come to mind . Could it be a permissions problem or even an issue with an apache directive.

Ill set up STB on my VPS server and see if that works.

Comment 25

[Paul Booker]

Worked perfectly first time on my VPS server!
 
I'll send some details later about my server configuration later today.

Thanks for the time on this Gozer.

Much appreciated.
Paul

Comment 26

[Paul Booker]

(In better english)

Worked perfectly first time on my VPS server!

I'll send some details later today about my server configuration.

Thanks for your time on this Gozer.

Much appreciated.
Paul

Comment 27

[Philippe M. Chiasson (:gozer)]

An upgrade to Drupal 6.6 did the trick. I've also moved the theme back to /themes and it's still working just fine. I'll file it as a Drupal 6.4 (or 6.5) bug.

Comment 28

[jamey]

Paul can we add 2 new image for otto and lucy vanguard
http://www.spreadthunderbird.com/user/142

http://www.spreadthunderbird.com/user/76

Comment 29

[jamey]

Attachment: lucy

Comment 30

[jamey]

Attachment: otto

Comment 31

[Philippe M. Chiasson (:gozer)]

Adding  (bin)  avatars/_142.gif
Adding  (bin)  avatars/_76.gif
Transmitting file data ..
Committed revision 26286.

Comment 32

[Philippe M. Chiasson (:gozer)]

pushed to staging and production.

in the future, better create new bugs instead of re-opening this one.