Skip to content

Commit 61e3ed1

Browse files
pieternnfx
authored andcommitted
Remove ID field from query and visualization structs
1 parent 1e394ee commit 61e3ed1

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

sqlanalytics/resource_query.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import (
1313

1414
// QueryEntity defines the parameters that can be set in the resource.
1515
type QueryEntity struct {
16-
ID string `json:"id,omitempty" tf:"computed"`
1716
DataSourceID string `json:"data_source_id"`
1817
Name string `json:"name"`
1918
Description string `json:"description,omitempty"`
@@ -143,7 +142,7 @@ func (q *QueryEntity) toAPIObject(schema map[string]*schema.Schema, data *schema
143142

144143
// Transform to API object.
145144
var aq api.Query
146-
aq.ID = q.ID
145+
aq.ID = data.Id()
147146
aq.DataSourceID = q.DataSourceID
148147
aq.Name = q.Name
149148
aq.Description = q.Description
@@ -244,7 +243,6 @@ func (q *QueryEntity) toAPIObject(schema map[string]*schema.Schema, data *schema
244243

245244
func (q *QueryEntity) fromAPIObject(aq *api.Query, schema map[string]*schema.Schema, data *schema.ResourceData) error {
246245
// Copy from API object.
247-
q.ID = aq.ID
248246
q.DataSourceID = aq.DataSourceID
249247
q.Name = aq.Name
250248
q.Description = aq.Description

sqlanalytics/resource_visualization.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import (
1515

1616
// VisualizationEntity defines the parameters that can be set in the resource.
1717
type VisualizationEntity struct {
18-
ID string `json:"id,omitempty" tf:"computed"`
1918
QueryID string `json:"query_id"`
2019
Type string `json:"type"`
2120
Name string `json:"name"`
@@ -32,8 +31,8 @@ func (v *VisualizationEntity) toAPIObject(schema map[string]*schema.Schema, data
3231
// Transform to API object.
3332
var av api.Visualization
3433

35-
if v.ID != "" {
36-
id, err := strconv.Atoi(v.ID)
34+
if data.Id() != "" {
35+
id, err := strconv.Atoi(data.Id())
3736
if err != nil {
3837
return nil, err
3938
}
@@ -50,7 +49,6 @@ func (v *VisualizationEntity) toAPIObject(schema map[string]*schema.Schema, data
5049

5150
func (v *VisualizationEntity) fromAPIObject(av *api.Visualization, schema map[string]*schema.Schema, data *schema.ResourceData) error {
5251
// Copy from API object.
53-
v.ID = strconv.Itoa(av.ID)
5452
v.QueryID = av.QueryID
5553
v.Type = strings.ToLower(av.Type)
5654
v.Name = av.Name

0 commit comments

Comments
 (0)