Fix import of bigquery transfer config location (GoogleCloudPlatform#…

…8768)

* Fix import of bigquery transfer config location

* Fix import of bigquery transfer config when name does not contain location

* Fix indentation in bigquery_data_transfer_self_link_as_name_set_location.go.erb

mmv1/products/bigquerydatatransfer/Config.yaml

@@ -31,7 +31,7 @@ custom_code: !ruby/object:Provider::Terraform::CustomCode
   constants: templates/terraform/constants/bigquery_data_transfer.go.erb
   decoder: templates/terraform/decoders/bigquery_data_transfer.go.erb
   encoder: templates/terraform/encoders/bigquery_data_transfer.go.erb
-  custom_import: templates/terraform/custom_import/self_link_as_name.erb
+  custom_import: templates/terraform/custom_import/bigquery_data_transfer_self_link_as_name_set_location.go.erb
   post_create: templates/terraform/post_create/set_computed_name.erb
 custom_diff: [
   'sensitiveParamCustomizeDiff',
@@ -74,8 +74,9 @@ properties:
     output: true
     description: |
       The resource name of the transfer config. Transfer config names have the
-      form projects/{projectId}/locations/{location}/transferConfigs/{configId}.
-      Where configId is usually a uuid, but this is not required.
+      form projects/{projectId}/locations/{location}/transferConfigs/{configId}
+      or projects/{projectId}/transferConfigs/{configId},
+      where configId is usually a uuid, but this is not required.
       The name is ignored when creating a transfer config.
   - !ruby/object:Api::Type::String
     name: 'destinationDatasetId'

mmv1/templates/terraform/custom_import/bigquery_data_transfer_self_link_as_name_set_location.go.erb

@@ -0,0 +1,21 @@
+
+config := meta.(*transport_tpg.Config)
+
+// current import_formats can't import fields with forward slashes in their value
+if err := tpgresource.ParseImportId([]string{"(?P<project>[^ ]+) (?P<name>[^ ]+)", "(?P<name>[^ ]+)"}, d, config); err != nil {
+	return nil, err
+}
+
+// import location if the name format follows: projects/{{project}}/locations/{{location}}/transferConfigs/{{config_id}}
+name := d.Get("name").(string)
+stringParts := strings.Split(name, "/")
+if len(stringParts) == 6 {
+	if err := d.Set("location", stringParts[3]); err != nil {
+		return nil, fmt.Errorf("Error setting location: %s", err)
+	}
+} else {
+	log.Printf("[INFO] Transfer config location not imported as it is not included in the name: %s", name)
+}
+
+
+return []*schema.ResourceData{d}, nil