Warn if getInitialProps is defined as as instance method

Ommitting the static keyword happens pretty often. Therefore we should warn the devloper in devmode. Closes: vercel#4782
HaNdTriX · Aug 8, 2018 · 0cfc658 · 0cfc658
1 parent 9018da1
commit 0cfc658
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/lib/utils.js b/lib/utils.js
@@ -54,6 +54,15 @@ export function isResSent (res) {
 }
 
 export async function loadGetInitialProps (Component, ctx) {
+  if (
+    process.env.NODE_ENV !== 'production' &&
+    Component.prototype &&
+    Component.prototype.getInitialProps
+  ) {
+    const compName = getDisplayName(Component)
+    console.warn(`"${compName}.getInitialProps()" is not defined as a static method and therefor ommited.`)
+  }
+
   if (!Component.getInitialProps) return {}
 
   const props = await Component.getInitialProps(ctx)