This is an issue on Pantheon and local installs. Another example of where the assumption that install.php will be run from sites/all/modules snuck back into the code. Not sure if this is something that can be caught with CiviCRM's unit testing.

CommentFileSizeAuthor
#3 1988296-logo-filename-change-2.patch530 byteskreynen
#2 1988296-logo-filename-change.patch530 byteskreynen
missing_logo.png169.56 KBkreynen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kreynen’s picture

kreynen CreditAttribution: kreynen commented
Status: Active » Fixed

Fixed by CiviCRM update. Image has been removed.

kreynen’s picture

kreynen CreditAttribution: kreynen commented
Status: Fixed » Needs review
FileSize
1988296-logo-filename-change.patch530 bytes

This actually wasn't fixed. I was just using Chrome so I wan't seeing the broken image icon. I'm not sure what is supposed to happen here, but the install/template.html includes...

<img src=<?php echo $installURLPath . "powered_by.png"?> />

There is a powered_by.png in i/powered_by.png, but on a default Drupal install the $installURLPath is empty. An empty $installURLPath works fine for...

<link rel="stylesheet" type="text/css" href=<?php echo $installURLPath . "template.css"?> />

There is a template.css in /install. There is a block_small.png which is really similar to powered_by.png and is not used anywhere else in CiviCRM AFAICT. This patch updates template.html to use block_small.png.

Cross posted at http://issues.civicrm.org/jira/browse/CRM-12670

kreynen’s picture

kreynen CreditAttribution: kreynen commented
FileSize
1988296-logo-filename-change-2.patch530 bytes
kreynen’s picture

kreynen CreditAttribution: kreynen commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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