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.

CommentFileSizeAuthor
#1 remove_sites_files_path.patch.txt4.2 KBjhuckabee
remove_sites_files_path.patch4.12 KBjhuckabee

Comments

jhuckabee’s picture

jhuckabee commented
StatusFileSize
new remove_sites_files_path.patch.txt4.2 KB

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

beatnikdude’s picture

beatnikdude commented

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

timb’s picture

timb commented

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

robloach’s picture

robloach
he/him
commented
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.

BenK’s picture

BenK commented

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

mrfelton’s picture

mrfelton commented
Status: Needs review » Reviewed & tested by the community

Works for me. Several good reviews here. Please lets get this committed. :)

eaton’s picture

eaton commented
Status: Reviewed & tested by the community » Fixed

Committted. Thanks!

Status: Fixed » Closed (fixed)

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