6.0.0-beta1
8/13/25

[#11517] Duplicate CSS files output
Summary Duplicate CSS files output
Queue Horde Base
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners Horde Developers (at) , mrubinsk (at) horde (dot) org
Requester mrubinsk (at) horde (dot) org
Created 10/13/2012 (4687 days ago)
Due
Updated 10/15/2012 (4685 days ago)
Assigned 10/14/2012 (4686 days ago)
Resolved 10/14/2012 (4686 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
10/15/2012 08:46:54 PM Git Commit Comment #7 Reply to this comment
Changes have been made in Git (develop):

commit 52a92f3eff8128a9751e7893b5de8e9629b31a53
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sat Oct 13 21:11:52 2012 -0400

     Bug: 11517  Don't request the 'default' theme's css files twice.

  framework/Core/lib/Horde/Themes/Cache.php |    4 ++--
  koward/www/horde/config/prefs.php.dist    |    2 +-
  2 files changed, 3 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/52a92f3eff8128a9751e7893b5de8e9629b31a53
10/14/2012 01:15:17 AM Michael Rubinsky Assigned to Michael Rubinsky
State ⇒ Resolved
 
10/14/2012 01:15:02 AM Michael Rubinsky Comment #6 Reply to this comment
Works perfectly fine for me.
E.g., from Kronolith:
This is what I see on the base kronolith page (dynamic view):
You were (probably) not seeing this because the default prefs.php 
value for theme was still bluewhite which does not exist. The code was 
adding the current theme AND the default theme files to the output so 
if you saved the theme pref at any point to 'default' then you would 
run into this.
10/14/2012 01:13:00 AM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit 52a92f3eff8128a9751e7893b5de8e9629b31a53
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Sat Oct 13 21:11:52 2012 -0400

     Bug: 11517  Don't request the 'default' theme's css files twice.

  framework/Core/lib/Horde/Themes/Cache.php |    4 ++--
  koward/www/horde/config/prefs.php.dist    |    2 +-
  2 files changed, 3 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/52a92f3eff8128a9751e7893b5de8e9629b31a53
10/14/2012 12:25:55 AM Michael Slusarz Comment #4 Reply to this comment
Works perfectly fine for me.
E.g., from Kronolith:
This is what I see on the base kronolith page (dynamic view):

<!--a75c305b1c0a6022--><link href="/horde/themes/default/screen.css" 
rel="stylesheet" type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/kronolith/themes/default/screen.css" rel="stylesheet" 
type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/kronolith/themes/default/dynamic/screen.css" 
rel="stylesheet" type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/themes/default/mozilla.css" rel="stylesheet" 
type="text/css" />
Maybe related to Bug: 11407 ?
I wouldn't think so, since I am not seeing it.  But maybe this is only 
triggered by a more complex share configuration since I don't use any 
of that.
10/14/2012 12:21:27 AM Michael Rubinsky Comment #3 Reply to this comment
Works perfectly fine for me.
E.g., from Kronolith:

<!--a75c305b1c0a6022--><link href="/horde/themes/default/screen.css" 
rel="stylesheet" type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/kronolith/themes/default/screen.css" rel="stylesheet" 
type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/themes/default/screen.css" rel="stylesheet" 
type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/kronolith/themes/default/screen.css" rel="stylesheet" 
type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/kronolith/themes/default/dynamic/screen.css" 
rel="stylesheet" type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/kronolith/themes/default/dynamic/screen.css" 
rel="stylesheet" type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/themes/default/mozilla.css" rel="stylesheet" 
type="text/css" /><!--a75c305b1c0a6022--><link 
href="/horde/themes/default/mozilla.css" rel="stylesheet" 
type="text/css" /


Maybe related to Bug: 11407 ?
10/14/2012 12:13:02 AM Michael Slusarz Comment #2
State ⇒ Feedback
Reply to this comment
Works perfectly fine for me.
10/13/2012 09:05:37 PM Michael Rubinsky Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Duplicate CSS files output
Queue ⇒ Horde Base
Assigned to Horde DevelopersHorde Developers
Milestone ⇒
Patch ⇒ No
State ⇒ Assigned
Reply to this comment
Applications' CSS files are currently output twice in both traditional 
and dynamic modes. Tested in IMP, Kronolith, Mnemo.

Saved Queries