Edgewall Software
Modify

Opened 17 years ago

Closed 15 years ago

Last modified 15 years ago

#3865 closed defect (fixed)

Document migration to site.html

Reported by: sam@… Owned by: shanec@…
Priority: normal Milestone: 0.11.5
Component: general Version: devel
Severity: normal Keywords: initenv inherit patch
Cc: m@…, trudolphi@…, rysiek@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description

The files in env/templates/... are not used, thus making CSS or templates tweaks per-environment impossible with latest Trac trunk. This is clearly a regression from previous revisions.

Attachments (1)

disable_env_site_template.patch (663 bytes ) - added by shanec@… 15 years ago.
only create site.html if one is not in shared template path

Download all attachments as: .zip

Change History (15)

comment:1 by Christian Boos, 17 years ago

Strange, seems to be still there, in Chrome's get_templates_dir.

comment:2 by Christopher Lenz, 17 years ago

Priority: highnormal
Severity: majornormal

Well, the Clearsilver site_*.cs are no longer used if the page is being rendered using Genshi. Instead, env/templates/site.html is being included if it's present.

We obviously need to add some upgrade instructions for guiding with the migration here.

in reply to:  2 comment:3 by Markus Tacker <m@…>, 17 years ago

Cc: m@… added

Replying to cmlenz:

We obviously need to add some upgrade instructions for guiding with the migration here.

Yes, please do so!

comment:5 by Christopher Lenz, 17 years ago

Keywords: documentation added
Owner: changed from Jonas Borgström to Christopher Lenz
Status: newassigned
Summary: User templates not loadedDocument migration to site.html

And this post, which would be a starting point for updates to TracInterfaceCustomization.

comment:6 by Markus Tacker <m@…>, 17 years ago

Cc: trudolphi@… added

comment:7 by Christian Boos, 17 years ago

Milestone: 0.11.10.11
Resolution: fixed
Status: assignedclosed

I believe this was fixed by r5703.

by shanec@…, 15 years ago

only create site.html if one is not in shared template path

comment:8 by shanec@…, 15 years ago

Resolution: fixed
Status: closedreopened

I ran into an issue where the shared template site.html *was not* being used. When creating an environment, a site.html file is written out into env/template. If I remove that, the shared site.html is used. The patch I just attached does not write out a env/template/site.html if one exists in the shared template dir.

comment:9 by Christian Boos, 15 years ago

Keywords: initenv inherit added; documentation removed
Milestone: 0.110.11.3

Seems in line with what we do for other settings when using the --inherit flag for creating new environments (TracAdmin#Notes).

Rescheduling to 0.11.3, for integrating attachment:disable_env_site_template.patch.

comment:10 by rysiek@…, 15 years ago

Cc: rysiek@… added

I second what shanec@… said - env/templates/site.html is clearly not needed (deleted site.html, had no global either), hence it's creation during initenv is superfluous - and as it causes problems when creating a global site.html file (I ran into the very same thing - created the global site.html and started debugging why isn't it used, only to find the default site.html in project env dirs).

I'd say it shouldn't be created in the first place anyway.

comment:11 by Christian Boos, 15 years ago

Keywords: patch added
Milestone: 0.11.60.11.5

comment:12 by Remy Blank, 15 years ago

Owner: changed from Christopher Lenz to Remy Blank
Status: reopenednew

comment:13 by Remy Blank, 15 years ago

Resolution: fixed
Status: newclosed

Patch applied in [8147]. Thanks!

comment:14 by Remy Blank, 15 years ago

Owner: changed from Remy Blank to shanec@…

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain shanec@….
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from shanec@… to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.