Overwrite option not working?

[ryexley]

I have the following configuration:

"style-image-refs": {
    src: ["app/assets/stylesheets/**/*.css"],
    overwrite: true,
    replacements: [
        { from: "url(\"/images/", to: "url(\"/assets/" },
        { from: "url(\"../images/", to: "url(\"/assets/" }
    ]
}

to replace image URL paths in CSS files. The overwrite attribute seems to be causing the replace functionality to break. If I modify the configuration to write the target files to a different directory, like this:

"style-image-refs": {
    src: ["app/assets/stylesheets/**/*.css"],
    dest: "app/assets/stylesheets/fixed/",
    replacements: [
        { from: "url(\"/images/", to: "url(\"/assets/" },
        { from: "url(\"../images/", to: "url(\"/assets/" }
    ]
}

the files get written to the fixed directory, and the target strings are replaced as expected. But when I use the overwrite attribute it doesn't work.

Anyone else experiencing this? Is my configuration setup wrong or something? Any help would be greatly appreciated.

Thanks.

[ryexley]

I'm pretty sure I can confirm this by looking at the source. The only place the overwrite option is referenced is in the replace function, and in the errorMessages object. It appears to be checked in the replace function, but it never actually gets passed on to the replaceFileMultiple function...so it doesn't appear to be getting used at all. Am I missing something?

[okcompewter]

Same here, the overwrite: true option gives an error:
Warning: Cannot read property 'slice' of undefined� Use --force to continue.

[ra-ryan]

@ryexley @okcompewter you've probably moved on by now but I figured I would share my results anyways. You can overwrite the target files by only including the src attribute and leaving out both the dest and overwrite attributes. This isn't how it is documented but it worked for me. So your original code sample would look like this:

"style-image-refs": {
    src: ["app/assets/stylesheets/**/*.css"],
    replacements: [
        { from: "url(\"/images/", to: "url(\"/assets/" },
        { from: "url(\"../images/", to: "url(\"/assets/" }
    ]
}

[Mifrill]

Use grunt-string-replace