use util.deprecate to deprecate properties in ContextModule

sokra · sokra · commit 47d00a525f81 · 2017-10-16T12:16:07.000+02:00
diff --git a/lib/ContextModule.js b/lib/ContextModule.js
@@ -4,6 +4,7 @@
 */
 "use strict";
 const path = require("path");
+const util = require("util");
 const Module = require("./Module");
 const OriginalSource = require("webpack-sources").OriginalSource;
 const RawSource = require("webpack-sources").RawSource;
@@ -23,32 +24,6 @@ class ContextModule extends Module {
 		this.built = false;
 	}
 
-	/* These getters are for backwards compatibility */
-	get recursive() {
-		console.error("contextModule.recursive has been moved to contextModule.options.recursive");
-		return this.options.recursive;
-	}
-
-	get regExp() {
-		console.error("contextModule.regExp has been moved to contextModule.options.regExp");
-		return this.options.regExp;
-	}
-
-	get addon() {
-		console.error("contextModule.addon has been moved to contextModule.options.addon");
-		return this.options.addon;
-	}
-
-	get async() {
-		console.error("contextModule.async has been moved to contextModule.options.async");
-		return this.options.async;
-	}
-
-	get chunkName() {
-		console.error("contextModule.chunkName has been moved to contextModule.options.chunkName");
-		return this.options.chunkName;
-	}
-
 	prettyRegExp(regexString) {
 		// remove the "/" at the front and the beginning
 		// "/foo/" -> "foo"
@@ -463,4 +438,54 @@ webpackEmptyAsyncContext.id = ${JSON.stringify(id)};`;
 	}
 }
 
+Object.defineProperty(ContextModule.prototype, "recursive", {
+	configurable: false,
+	get: util.deprecate(function() {
+		return this.options.recursive;
+	}, "ContextModule.recursive has been moved to ContextModule.options.recursive"),
+	set: util.deprecate(function(value) {
+		this.options.recursive = value;
+	}, "ContextModule.recursive has been moved to ContextModule.options.recursive")
+});
+
+Object.defineProperty(ContextModule.prototype, "regExp", {
+	configurable: false,
+	get: util.deprecate(function() {
+		return this.options.regExp;
+	}, "ContextModule.regExp has been moved to ContextModule.options.regExp"),
+	set: util.deprecate(function(value) {
+		this.options.regExp = value;
+	}, "ContextModule.regExp has been moved to ContextModule.options.regExp")
+});
+
+Object.defineProperty(ContextModule.prototype, "addon", {
+	configurable: false,
+	get: util.deprecate(function() {
+		return this.options.addon;
+	}, "ContextModule.addon has been moved to ContextModule.options.addon"),
+	set: util.deprecate(function(value) {
+		this.options.addon = value;
+	}, "ContextModule.addon has been moved to ContextModule.options.addon")
+});
+
+Object.defineProperty(ContextModule.prototype, "async", {
+	configurable: false,
+	get: util.deprecate(function() {
+		return this.options.async;
+	}, "ContextModule.async has been moved to ContextModule.options.async"),
+	set: util.deprecate(function(value) {
+		this.options.async = value;
+	}, "ContextModule.async has been moved to ContextModule.options.async")
+});
+
+Object.defineProperty(ContextModule.prototype, "chunkName", {
+	configurable: false,
+	get: util.deprecate(function() {
+		return this.options.chunkName;
+	}, "ContextModule.chunkName has been moved to ContextModule.options.chunkName"),
+	set: util.deprecate(function(value) {
+		this.options.chunkName = value;
+	}, "ContextModule.chunkName has been moved to ContextModule.options.chunkName")
+});
+
 module.exports = ContextModule;
