Object.hasOwn & hasOwnProperty replace with lodash has (#367)

* Object.hasOwn & hasOwnProperty replace with lodash has

* fix _has
This commit is contained in:
Chirag Chhatrala 2024-04-03 15:19:33 +05:30 committed by GitHub
parent 615ac88f89
commit a7517252c1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
13 changed files with 37 additions and 22 deletions

View File

@ -2,6 +2,7 @@ import { ref, computed, watch } from 'vue'
import { themes } from '~/lib/forms/form-themes.js' import { themes } from '~/lib/forms/form-themes.js'
import {default as _get} from 'lodash/get' import {default as _get} from 'lodash/get'
import {default as _set} from 'lodash/set' import {default as _set} from 'lodash/set'
import { default as _has } from 'lodash/has'
export const inputProps = { export const inputProps = {
id: { type: String, default: null }, id: { type: String, default: null },
@ -31,7 +32,7 @@ export function useFormInput (props, context, formPrefixKey = null) {
}) })
const hasValidation = computed(() => { const hasValidation = computed(() => {
return props.form !== null && props.form !== undefined && props.form.hasOwnProperty('errors') return props.form !== null && props.form !== undefined && _has(props.form, 'errors')
}) })
const hasError = computed(() => { const hasError = computed(() => {

View File

@ -7,6 +7,7 @@
</template> </template>
<script setup> <script setup>
import { default as _has } from 'lodash/has'
const props = defineProps({ const props = defineProps({
color: { color: {
@ -36,7 +37,7 @@ const iconBaseClasses = {
} }
const activeColor = computed(() => { const activeColor = computed(() => {
return Object.hasOwn(baseClasses, props.color) ? props.color : 'gray' return _has(baseClasses, props.color) ? props.color : 'gray'
}) })
const classes = computed(() => { const classes = computed(() => {

View File

@ -20,9 +20,9 @@
</button> </button>
</div> </div>
<div class="flex border-b pb-4" <div class="flex border-b pb-4"
v-if="$slots.hasOwnProperty('icon') || $slots.hasOwnProperty('title')" v-if="_has($slots,'icon') || _has($slots,'title')"
:class="[{'flex-col sm:items-start':!compactHeader, 'items-center justify-center py-6 gap-x-4':compactHeader},headerInnerPadding]"> :class="[{'flex-col sm:items-start':!compactHeader, 'items-center justify-center py-6 gap-x-4':compactHeader},headerInnerPadding]">
<div v-if="$slots.hasOwnProperty('icon')" :class="{'w-full mb-4 flex justify-center':!compactHeader}"> <div v-if="_has($slots,'icon')" :class="{'w-full mb-4 flex justify-center':!compactHeader}">
<div class="w-14 h-14 rounded-full flex justify-center items-center" <div class="w-14 h-14 rounded-full flex justify-center items-center"
:class="'bg-'+iconColor+'-100 text-'+iconColor+'-600'" :class="'bg-'+iconColor+'-100 text-'+iconColor+'-600'"
> >
@ -30,7 +30,7 @@
</div> </div>
</div> </div>
<div class="mt-3 text-center sm:mt-0" :class="{'w-full':!compactHeader}"> <div class="mt-3 text-center sm:mt-0" :class="{'w-full':!compactHeader}">
<h2 v-if="$slots.hasOwnProperty('title')" <h2 v-if="_has($slots,'title')"
class="text-2xl font-semibold text-center text-gray-900" class="text-2xl font-semibold text-center text-gray-900"
> >
<slot name="title"/> <slot name="title"/>
@ -42,7 +42,7 @@
<slot/> <slot/>
</div> </div>
<div v-if="$slots.hasOwnProperty('footer')" class="bg-gray-50 border-t rounded-b-xl text-right" :class="footerInnerPadding"> <div v-if="_has($slots,'footer')" class="bg-gray-50 border-t rounded-b-xl text-right" :class="footerInnerPadding">
<slot name="footer"/> <slot name="footer"/>
</div> </div>
</div> </div>
@ -53,6 +53,7 @@
<script setup> <script setup>
import {watch} from "vue"; import {watch} from "vue";
import { default as _has } from 'lodash/has'
const props = defineProps({ const props = defineProps({
show: { show: {

View File

@ -118,6 +118,7 @@ import FormCleanings from '../../pages/forms/show/FormCleanings.vue'
import VTransition from '~/components/global/transitions/VTransition.vue' import VTransition from '~/components/global/transitions/VTransition.vue'
import {pendingSubmission} from "~/composables/forms/pendingSubmission.js"; import {pendingSubmission} from "~/composables/forms/pendingSubmission.js";
import clonedeep from "clone-deep"; import clonedeep from "clone-deep";
import { default as _has } from 'lodash/has'
export default { export default {
components: { VTransition, VButton, OpenFormButton, OpenForm, FormCleanings }, components: { VTransition, VButton, OpenFormButton, OpenForm, FormCleanings },
@ -155,7 +156,7 @@ export default {
return import.meta.client && window.location.href.includes('popup=true') return import.meta.client && window.location.href.includes('popup=true')
}, },
theme () { theme () {
return this.themes[this.themes.hasOwnProperty(this.form.theme) ? this.form.theme : 'default'] return this.themes[_has(this.themes, this.form.theme) ? this.form.theme : 'default']
}, },
isPublicFormPage () { isPublicFormPage () {
return this.$route.name === 'forms-slug' return this.$route.name === 'forms-slug'

View File

@ -56,6 +56,7 @@
<script> <script>
import { computed } from 'vue' import { computed } from 'vue'
import FormLogicPropertyResolver from "~/lib/forms/FormLogicPropertyResolver.js" import FormLogicPropertyResolver from "~/lib/forms/FormLogicPropertyResolver.js"
import { default as _has } from 'lodash/has'
export default { export default {
name: 'OpenFormField', name: 'OpenFormField',
@ -248,7 +249,7 @@ export default {
} }
if (['select', 'multi_select'].includes(field.type)) { if (['select', 'multi_select'].includes(field.type)) {
inputProperties.options = (field.hasOwnProperty(field.type)) inputProperties.options = (_has(field, field.type))
? field[field.type].options.map(option => { ? field[field.type].options.map(option => {
return { return {
name: option.name, name: option.name,

View File

@ -19,6 +19,7 @@
<script setup> <script setup>
import { defineProps, computed } from 'vue' import { defineProps, computed } from 'vue'
import { default as _has } from 'lodash/has'
const { copy } = useClipboard() const { copy } = useClipboard()
const props = defineProps({ const props = defineProps({
@ -40,7 +41,7 @@ const preFillUrl = computed(() => {
const url = props.form.share_url const url = props.form.share_url
const uriComponents = new URLSearchParams() const uriComponents = new URLSearchParams()
props.form.properties.filter((property) => { props.form.properties.filter((property) => {
return props.formData.hasOwnProperty(property.id) && props.formData[property.id] !== null return _has(props.formData, property.id) && props.formData[property.id] !== null
}).forEach((property) => { }).forEach((property) => {
if (Array.isArray(props.formData[property.id])) { if (Array.isArray(props.formData[property.id])) {
props.formData[property.id].forEach((value) => { props.formData[property.id].forEach((value) => {

View File

@ -62,6 +62,7 @@
import VSwitch from '../../../../forms/components/VSwitch.vue' import VSwitch from '../../../../forms/components/VSwitch.vue'
import OpenCompleteForm from '../../OpenCompleteForm.vue' import OpenCompleteForm from '../../OpenCompleteForm.vue'
import {handleDarkMode} from "~/lib/forms/public-page.js" import {handleDarkMode} from "~/lib/forms/public-page.js"
import { default as _has } from 'lodash/has'
export default { export default {
components: { OpenCompleteForm, VSwitch }, components: { OpenCompleteForm, VSwitch },
@ -89,7 +90,7 @@ export default {
} }
}, },
creating () { // returns true if we are creating a form creating () { // returns true if we are creating a form
return !this.form.hasOwnProperty('id') return !_has(this.form, 'id')
} }
}, },

View File

@ -66,10 +66,10 @@
</template> </template>
<script> <script>
import { computed } from 'vue'
import clonedeep from 'clone-deep' import clonedeep from 'clone-deep'
import EditorOptionsPanel from '../../../editors/EditorOptionsPanel.vue' import EditorOptionsPanel from '../../../editors/EditorOptionsPanel.vue'
import SelectInput from '../../../../forms/SelectInput.vue' import SelectInput from '../../../../forms/SelectInput.vue'
import { default as _has } from 'lodash/has'
export default { export default {
components: { SelectInput, EditorOptionsPanel }, components: { SelectInput, EditorOptionsPanel },
@ -152,7 +152,7 @@ export default {
['title', 'description', 'properties', 'cleanings', 'views_count', 'submissions_count', 'workspace', 'workspace_id', 'updated_at', ['title', 'description', 'properties', 'cleanings', 'views_count', 'submissions_count', 'workspace', 'workspace_id', 'updated_at',
'share_url', 'slug', 'notion_database_url', 'id', 'database_id', 'database_fields_update', 'creator', 'share_url', 'slug', 'notion_database_url', 'id', 'database_id', 'database_fields_update', 'creator',
'created_at', 'deleted_at', 'last_edited_human'].forEach((property) => { 'created_at', 'deleted_at', 'last_edited_human'].forEach((property) => {
if (copyForm.hasOwnProperty(property)) { if (_has(copyForm, property)) {
delete copyForm[property] delete copyForm[property]
} }
}) })

View File

@ -70,6 +70,7 @@
import ConditionEditor from './ConditionEditor.client.vue' import ConditionEditor from './ConditionEditor.client.vue'
import Modal from '../../../../global/Modal.vue' import Modal from '../../../../global/Modal.vue'
import clonedeep from 'clone-deep' import clonedeep from 'clone-deep'
import { default as _has } from 'lodash/has'
export default { export default {
name: 'FormBlockLogicEditor', name: 'FormBlockLogicEditor',
@ -100,7 +101,7 @@ export default {
computed: { computed: {
copyFromOptions () { copyFromOptions () {
return this.form.properties.filter((field) => { return this.form.properties.filter((field) => {
return field.id !== this.field.id && field.hasOwnProperty('logic') && field.logic !== null && field.logic !== {} return field.id !== this.field.id && _has(field, 'logic') && field.logic !== null && field.logic !== {}
}).map((field) => { }).map((field) => {
return { name: field.name, value: field.id } return { name: field.name, value: field.id }
}) })
@ -166,7 +167,7 @@ export default {
}, },
mounted () { mounted () {
if (!this.field.hasOwnProperty('logic')) { if (!_has(this.field, 'logic')) {
this.field.logic = this.logic this.field.logic = this.logic
} }
}, },

View File

@ -316,6 +316,7 @@ import timezones from '~/data/timezones.json'
import countryCodes from '~/data/country_codes.json' import countryCodes from '~/data/country_codes.json'
import CountryFlag from 'vue-country-flag-next' import CountryFlag from 'vue-country-flag-next'
import FormBlockLogicEditor from '../../components/form-logic-components/FormBlockLogicEditor.vue' import FormBlockLogicEditor from '../../components/form-logic-components/FormBlockLogicEditor.vue'
import { default as _has } from 'lodash/has'
export default { export default {
name: 'FieldOptions', name: 'FieldOptions',
@ -556,7 +557,7 @@ export default {
} }
if (this.field.type in defaultFieldValues) { if (this.field.type in defaultFieldValues) {
Object.keys(defaultFieldValues[this.field.type]).forEach(key => { Object.keys(defaultFieldValues[this.field.type]).forEach(key => {
if (!Object.hasOwn(this.field,key)) { if (!_has(this.field,key)) {
this.field[key] = defaultFieldValues[this.field.type][key] this.field[key] = defaultFieldValues[this.field.type][key]
} }
}) })

View File

@ -21,7 +21,7 @@
</tr> </tr>
</thead> </thead>
<tbody v-if="data.length > 0" class="n-table-body bg-white dark:bg-notion-dark-light"> <tbody v-if="data.length > 0" class="n-table-body bg-white dark:bg-notion-dark-light">
<tr v-if="$slots.hasOwnProperty('actions')" :id="'table-actions-' + tableHash" ref="actions-row" <tr v-if="objectHas($slots,'actions')" :id="'table-actions-' + tableHash" ref="actions-row"
class="action-row absolute w-full" style="will-change: transform; transform: translate3d(0px, 32px, 0px)"> class="action-row absolute w-full" style="will-change: transform; transform: translate3d(0px, 32px, 0px)">
<td :colspan="columns.length" class="p-1"> <td :colspan="columns.length" class="p-1">
<slot name="actions" /> <slot name="actions" />
@ -71,6 +71,7 @@ import ResizableTh from './components/ResizableTh.vue'
import RecordOperations from '../components/RecordOperations.vue' import RecordOperations from '../components/RecordOperations.vue'
import clonedeep from 'clone-deep' import clonedeep from 'clone-deep'
import { hash } from "~/lib/utils.js"; import { hash } from "~/lib/utils.js";
import { default as _has } from 'lodash/has'
export default { export default {
components: { ResizableTh, RecordOperations }, components: { ResizableTh, RecordOperations },
@ -174,18 +175,18 @@ export default {
// Column color // Column color
colColor = null colColor = null
if (!col.hasOwnProperty('color') || col.color === 'default') { if (!_has(col, 'color') || col.color === 'default') {
colColor = 'text-gray-700 dark:text-gray-300' colColor = 'text-gray-700 dark:text-gray-300'
} }
colColor = `text-${col.color}` colColor = `text-${col.color}`
// Column font weight // Column font weight
if (col.hasOwnProperty('bold') && col.bold) { if (_has(col, 'bold') && col.bold) {
colFontWeight = 'font-semibold' colFontWeight = 'font-semibold'
} }
// Column wrapping // Column wrapping
if (!col.hasOwnProperty('wrap_text') || !col.wrap_text) { if (!_has(col, 'wrap_text') || !col.wrap_text) {
colWrap = 'truncate' colWrap = 'truncate'
} }
@ -195,7 +196,7 @@ export default {
if (this.internalColumns) { if (this.internalColumns) {
this.$nextTick(() => { this.$nextTick(() => {
this.internalColumns.forEach(col => { this.internalColumns.forEach(col => {
if (!col.hasOwnProperty('cell_width')) { if (!_has(col, 'cell_width')) {
if (this.allowResize && this.internalColumns.length && document.getElementById('table-head-cell-' + col.id)) { if (this.allowResize && this.internalColumns.length && document.getElementById('table-head-cell-' + col.id)) {
// Within editor // Within editor
this.resizeCol(col, document.getElementById('table-head-cell-' + col.id).offsetWidth) this.resizeCol(col, document.getElementById('table-head-cell-' + col.id).offsetWidth)
@ -256,6 +257,9 @@ export default {
}, },
setColumns(val) { setColumns(val) {
this.$emit('update-columns', val) this.$emit('update-columns', val)
},
objectHas(object, key) {
return _has(object, key)
} }
} }
} }

View File

@ -1,6 +1,7 @@
<template></template> <template></template>
<script setup> <script setup>
import {onMounted} from "vue"; import {onMounted} from "vue";
import { default as _has } from 'lodash/has'
const scriptLoaded = ref(false); const scriptLoaded = ref(false);
const authStore = useAuthStore() const authStore = useAuthStore()
@ -52,7 +53,7 @@ onMounted(() => {
if (import.meta.server) return if (import.meta.server) return
// Setup base // Setup base
if (!window.hasOwnProperty('Featurebase') || typeof window.Featurebase !== "function") { if (!_has(window, 'Featurebase') || typeof window.Featurebase !== "function") {
window.Featurebase = function () { window.Featurebase = function () {
(window.Featurebase.q = window.Featurebase.q || []).push(arguments); (window.Featurebase.q = window.Featurebase.q || []).push(arguments);
}; };

View File

@ -1,4 +1,5 @@
import { themes } from '~/lib/forms/form-themes.js' import { themes } from '~/lib/forms/form-themes.js'
import { default as _has } from 'lodash/has'
export default { export default {
props: { props: {
@ -31,7 +32,7 @@ export default {
} }
}, },
hasValidation () { hasValidation () {
return this.form !== null && this.form !== undefined && this.form.hasOwnProperty('errors') return this.form !== null && this.form !== undefined && _has(this.form, 'errors')
}, },
compVal: { compVal: {
set (val) { set (val) {