Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bad condition: modal form for datagrid row editing not populated #471

Merged
merged 2 commits into from
Oct 29, 2013

Conversation

jean
Copy link
Member

@jean jean commented Oct 28, 2013

If we have a tempdoc, it will already have been populated from Plomino_datagrid_rowdata.

I was tripped up by the logic in BaseField.getFieldValue.
It looks like this isn't being caught by tests.

If we have a tempdoc, it will already have been populated from
Plomino_datagrid_rowdata.
@coveralls
Copy link

Coverage Status

Coverage decreased (-53.01%) when pulling afec8c9 on jean-reorder-validation into 6bbb1e6 on github-main.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) when pulling 53efca6 on jean-reorder-validation into 6bbb1e6 on github-main.

ebrehault added a commit that referenced this pull request Oct 29, 2013
Bad condition: modal form for datagrid row editing not populated
@ebrehault ebrehault merged commit 5eee407 into github-main Oct 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants