Skip to content
This repository was archived by the owner on Oct 9, 2020. It is now read-only.

fix: loaderUtils.parseQuery() deprecation warning (loader-utils) #57

Closed
wants to merge 26 commits into from

Conversation

mitkaaa
Copy link

@mitkaaa mitkaaa commented Apr 17, 2017

Special fix for the latest version of the loader-utils(^1.1.0)
Removes warning when transpiling the webpack

Jaeho Lee and others added 22 commits September 16, 2016 18:54
add missing idPrefix as default value to config
Multiple classses in class attribute not prefixed when using classPrefix
multiple classes in class string, fix for Issue webpack-contrib#39
For CSS properties that contain '.' in the value.
[fix] don't transform webpack2 query objects
@jsf-clabot
Copy link

jsf-clabot commented Apr 17, 2017

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 10 committers have signed the CLA.

✅ sairion
✅ d3viant0ne
✅ mitkaaa
❌ Jay Jaeho Lee
❌ noxan
❌ nonmanifold
❌ alok pepakayala
❌ kinetifex
❌ Stuk
❌ =


Jay Jaeho Lee, alok pepakayala, = seem not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

.travis.yml Outdated
@@ -0,0 +1,6 @@
sudo: false
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't add .travis.yml in this PR, we use webpack-defaults for all repositories https://github.com/webpack-contrib/webpack-defaults

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@joshwiens joshwiens changed the title fix for loader-utils refactor: Update to latest loader-utils Jul 12, 2017
index.js Outdated
@@ -59,10 +59,7 @@ function getExtractedSVG(svgStr, query) {
function SVGInlineLoader(content) {
this.cacheable && this.cacheable();
this.value = content;
// Configuration
var query = loaderUtils.parseQuery(this.query);

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should still be required. var query = loaderUtils.getOptions(this) || {};

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

const options = loaderUtils.getOptions(this) && global rename query => options

index.js Outdated
@@ -59,10 +59,7 @@ function getExtractedSVG(svgStr, query) {
function SVGInlineLoader(content) {
this.cacheable && this.cacheable();
this.value = content;
// Configuration
var query = loaderUtils.parseQuery(this.query);

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

const options = loaderUtils.getOptions(this) && global rename query => options

@michael-ciniawsky michael-ciniawsky changed the title refactor: Update to latest loader-utils fix: loaderUtils.parseQuery() deprecation warning (loader-utils) Jul 14, 2017
@mitkaaa
Copy link
Author

mitkaaa commented Aug 7, 2017

I checked it works :)

@joshwiens
Copy link
Member

This is going to need to be rebased

@le0m
Copy link

le0m commented Dec 19, 2017

is there any update on this?
(cross-reference: #72)

@joshwiens
Copy link
Member

@le0m - I'm going to take care of this in a separate PR as the loader utils change is not backwards compatible.

With Webpack 4 on the way, we are a bit buried at the moment, i'll try and get a beta build that handles the deprecation warning & Webpack 4 support before the end of the week.

@jhnns
Copy link
Member

jhnns commented Feb 7, 2020

Fixed by https://github.com/webpack-contrib/svg-inline-loader/pull/72/files

@jhnns jhnns closed this Feb 7, 2020
@vidal7
Copy link

vidal7 commented Feb 7, 2020

Yeah! Thank you. @jhnns, I supposed that there will be a release soon on NPM registry? I don't see it on NPM yet.

@vidal7
Copy link

vidal7 commented Feb 10, 2020

@jhnns, I still don't see the release on npmjs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.