apache / apache/grails-core

AutoTimestampEventListener should only set dateCreated if it is null

Open
#14,662 1 comment 0 reactions 0 assignees View on GitHub
relates-to: gorm type: improvement
Dominant language
Groovy
Stars
2.9k
Forks
975
Avg merge
1d 22h
Merged PRs (30d)
92

Description

Currently it overwrites whatever value is in the field. This complicates integration testing as it makes it hard to populate the database with known data using GORM.

Contributor guide

Open the contributing guide

Research direction

Locate the AutoTimestampEventListener entry point in the GORM-related code and inspect how it handles dateCreated. Add coverage for an already populated dateCreated value, then verify that the existing value is preserved while a null value is still assigned a timestamp.

Written by the indexing model from the issue text.

Assessment

Tech stack
groovy
Domain
database
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.