Some time ago I've noticed that production site is missing Pixture Reloaded header logo and other pictures. These links result in 404:

http://mydomain.lt/sites/default/files/advagg_css/color/pixture_reloaded-a34c302b/bullet-round.png
http://mydomain.lt/sites/default/files/advagg_css/color/pixture_reloaded-a34c302b/header.png
http://mydomain.lt/sites/default/files/advagg_css/color/pixture_reloaded-a34c302b/header-tile.png
http://mydomain.lt/sites/default/files/advagg_css/color/pixture_reloaded-a34c302b/bg-wall.png

Tried clearing cashes and so forth, but cannot get it fixed.

Actual files are not in `sites/default/files/advagg_css` directory, but in `sites/default/files/color/pixture_reloaded-971003ce`, so I assume advagg fails somewhere generating final CSS, or something...

To reproduce on clean install:

1. Install currenty latest Drupal 7.56.
2. Install Adaptive Theme 7.x-3.4 https://ftp.drupal.org/files/projects/adaptivetheme-7.x-3.4.tar.gz and Pixture Reloaded 7.x-3.1 https://ftp.drupal.org/files/projects/pixture_reloaded-7.x-3.1.tar.gz . Enable Pixture Reloaded as default theme, select some color ("Teal" for example).
3. Install older AdvAgg version: advagg 7.x-2.23 https://ftp.drupal.org/files/projects/advagg-7.x-2.23.tar.gz
4. Update to advagg 7.x-2.24 https://ftp.drupal.org/files/projects/advagg-7.x-2.24.tar.gz . Some errors are shown during update.php.

At this point, Pixture Reloaded header is already missing.

5. Update to latest advagg 7.x-2.25 https://ftp.drupal.org/files/projects/advagg-7.x-2.25.tar.gz . Header is still missing.

I did some adjustments to Drupal performance and AdvAgg settings along with Pixture Reloaded options to match production ones. If you cannot reproduce on your clean install install, I can actually send (attach) a copy of my test-case Drupal installation including SQL dump.

?advagg-debug=1 output is attached.

CommentFileSizeAuthor
#11 advagg-2892183-11-add-in-color-test.patch3.97 KBmikeytown2
#8 advagg-2892183-8-handle-local-or-stream-wrappers-css.patch15.5 KBmikeytown2
#7 advagg-2892183-7-handle-local-or-stream-wrappers-css.patch2.07 KBmikeytown2
#4 advagg-2892183-5-fix-public-css-location.patch994 bytesmikeytown2
#3 advagg-2892183-3-fix-public-css-location.patch687 bytesmikeytown2
advagg-debug.txt185.62 KBTalkless
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Talkless created an issue. See original summary.

Talkless’s picture

Talkless CreditAttribution: Talkless as a volunteer commented
Issue summary: View changes
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Active » Needs review
FileSize
advagg-2892183-3-fix-public-css-location.patch687 bytes

Found the issue. Will take a little while add tests and things like that. This should stop the bleeding.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Version: 7.x-2.25 » 7.x-2.x-dev
FileSize
advagg-2892183-5-fix-public-css-location.patch994 bytes

  • mikeytown2 committed 625486f on 7.x-2.x 
    Issue #2892183 by mikeytown2, Talkless: 404 for Pixture Reloaded theme...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Active
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Active » Needs review
FileSize
advagg-2892183-7-handle-local-or-stream-wrappers-css.patch2.07 KB

Still need tests; worked on getting the core functionality to handle just about any stream wrapper input.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
FileSize
advagg-2892183-8-handle-local-or-stream-wrappers-css.patch15.5 KB

Still need to add in some tests but this should fix the issue with the current test failure.

  • mikeytown2 committed 3d32dc2 on 7.x-2.x 
    Issue #2892183 by mikeytown2: 404 for Pixture Reloaded theme files (...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Active

Now going to work on tests.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Active » Needs review
FileSize
advagg-2892183-11-add-in-color-test.patch3.97 KB

This is the test

  • mikeytown2 committed b2b1437 on 7.x-2.x 
    Issue #2892183 by mikeytown2: 404 for Pixture Reloaded theme files (...
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Fixed
Talkless’s picture

Talkless CreditAttribution: Talkless as a volunteer commented

Thanks. Is there known time frame for next AdvAgg release?

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented

I try to release a new one when all bug reports in the issue queue are "fixed". I'm hoping to release one by Friday or Monday.

Status: Fixed » Closed (fixed)

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