Remove hard-coded file directory path from database. [#341832]

CSS Injector is currently storing the file name including the site's files directory path in the database for a given rule. I'm working on a project where we do a lot of copying of various sites for testing and development purposes where the files directory path is never the same. Every time we copy a site we have to either go update the database with the correct path, or manually reload the styles in the admin interface.

I'm submitting a patch that removes the file_path field from the database all together. Please consider merging these changes into the project.

Comment	File	Size	Author
#1	remove_sites_files_path.patch.txt	4.2 KB	jhuckabee
	remove_sites_files_path.patch	4.12 KB	jhuckabee

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

jhuckabee CreditAttribution: jhuckabee commented 4 December 2008 at 16:06

File	Size
remove_sites_files_path.patch.txt	4.2 KB

Previous patch had an issue with saving new rules. Attaching a new patch.

Comment #2

BeatnikDude CreditAttribution: BeatnikDude commented 19 May 2009 at 20:27

I have yet to try the patch, but will. It is a good idea as stated.

Comment #3

timb CreditAttribution: timb commented 20 May 2009 at 06:45

The patch works for me. I second the idea of integrating it into the module.

Comment #4

RobLoach

he/him

CreditAttribution: RobLoach commented 17 June 2009 at 18:08

Priority:

Normal

» Critical

I ran into this when migrating over a site. I'll test this with the upgrade path as well. Pushing to critical because it breaks sites during migration.

Comment #5

BenK CreditAttribution: BenK commented 29 October 2009 at 02:25

The patch shown in #1 works perfectly for me. This functionality is very much needed for anyone who ever moves a site from a development server to a production server (and changes the file directory path in the process).

Can we get this committed to the module ASAP?

Cheers,
Ben