Skip to content

Commit

Permalink
Merge pull request #3 from plexsysio/update.2
Browse files Browse the repository at this point in the history
fix: minor fix in autoencoding
  • Loading branch information
aloknerurkar committed Dec 4, 2021
2 parents aa2812d + 12630d5 commit e2f4fbf
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion autoencoding/autoencoding.go
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,12 @@ func (i *item) GetID() string {

func (i *item) SetID(id string) {
v := reflect.ValueOf(i.val).Elem()
v.FieldByName(i.id).SetString(id)
// Ideally there should be some tag field to specify if we should use this
// setter or not. For now, we will check if ID was set already, we will use
// that, else set
if v.FieldByName(i.id).String() == "" {
v.FieldByName(i.id).SetString(id)
}
}

func (i *item) Get() interface{} {
Expand Down

0 comments on commit e2f4fbf

Please sign in to comment.