CodeMirror/budibase
1
0
Fork 0
mirror of synced 2024-09-09 22:16:26 +12:00
Code Issues Projects Releases Wiki Activity

Merge pull request #13752 from Budibase/BUDI-8279/prevent-changing-date-or-time-only-when-fetched

Browse source 
Prevent changing date or time only when fetched
This commit is contained in:
Adria Navarro 2024-05-23 15:27:26 +02:00 committed by GitHub
commit 7d77b517be
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 31 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

40
packages/builder/src/components/backend/DataTable/modals/CreateEditColumn.svelte
View file

@ -604,27 +604,29 @@
<DatePicker bind:value={editableColumn.constraints.datetime.latest} /> <DatePicker bind:value={editableColumn.constraints.datetime.latest} />
</div> </div>
</div> </div>
{#if datasource?.source !== SourceName.ORACLE && datasource?.source !== SourceName.SQL_SERVER && !editableColumn.dateOnly} {#if !editableColumn.timeOnly}
<div> {#if datasource?.source !== SourceName.ORACLE && datasource?.source !== SourceName.SQL_SERVER && !editableColumn.dateOnly}
<div class="row"> <div>
<Label>Time zones</Label> <div class="row">
<AbsTooltip <Label>Time zones</Label>
position="top" <AbsTooltip
type="info" position="top"
text={isCreating type="info"
? null text={isCreating
: "We recommend not changing how timezones are handled for existing columns, as existing data will not be updated"} ? null
> : "We recommend not changing how timezones are handled for existing columns, as existing data will not be updated"}
<Icon size="XS" name="InfoOutline" /> >
</AbsTooltip> <Icon size="XS" name="InfoOutline" />
</AbsTooltip>
</div>
<Toggle
bind:value={editableColumn.ignoreTimezones}
text="Ignore time zones"
/>
</div> </div>
<Toggle {/if}
bind:value={editableColumn.ignoreTimezones} <Toggle bind:value={editableColumn.dateOnly} text="Date only" />
text="Ignore time zones"
/>
</div>
{/if} {/if}
<Toggle bind:value={editableColumn.dateOnly} text="Date only" />
{:else if editableColumn.type === FieldType.NUMBER && !editableColumn.autocolumn} {:else if editableColumn.type === FieldType.NUMBER && !editableColumn.autocolumn}
<div class="split-label"> <div class="split-label">
<div class="label-length"> <div class="label-length">

10
packages/server/src/sdk/app/tables/external/index.ts vendored
View file

@ -73,6 +73,16 @@ function validate(table: Table, oldTable?: Table) {
`Column "${key}" has subtype "${column.subtype}" - this is not supported.` `Column "${key}" has subtype "${column.subtype}" - this is not supported.`
) )
} }
if (column.type === FieldType.DATETIME) {
const oldColumn = oldTable?.schema[key] as typeof column
if (oldColumn && column.timeOnly !== oldColumn.timeOnly) {
throw new Error(
`Column "${key}" can not change from time to datetime or viceversa.`
)
}
}
} }
} }