Merge pull request #13308 from Budibase/reenable-no-prototype-builtins

Reenable no-prototype-builtins

.eslintrc.json

@@ -42,7 +42,6 @@
       },
       "rules": {
         "no-unused-vars": "off",
-        "no-prototype-builtins": "off",
         "local-rules/no-budibase-imports": "error"
       }
     },
@@ -60,7 +59,6 @@
       },
       "rules": {
         "no-unused-vars": "off",
-        "no-prototype-builtins": "off",
         "local-rules/no-test-com": "error",
         "local-rules/email-domain-example-com": "error",
         "no-console": "warn",

packages/server/src/integrations/utils.ts

@@ -330,7 +330,7 @@ function copyExistingPropsOver(
 
     const existingTableSchema = entities[tableName].schema
     for (let key in existingTableSchema) {
-      if (!existingTableSchema.hasOwnProperty(key)) {
+      if (!Object.prototype.hasOwnProperty.call(existingTableSchema, key)) {
         continue
       }
       const column = existingTableSchema[key]