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.

Sign up for GitHub

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

Jump to bottom

SSOM - Fix DefaultFormula bug, add specific tests for DefaultFormula property #855

Closed
SubPointSupport opened this issue Jul 19, 2016 · 0 comments
Closed

SSOM - Fix DefaultFormula bug, add specific tests for DefaultFormula property #855

SubPointSupport opened this issue Jul 19, 2016 · 0 comments
Assignees
@SubPointSupport
Labels
bug community-feedback contribution-easy
Milestone
2016-08-01

Comments

@SubPointSupport
Copy link
Contributor

SSOM field model handler has the following typo/bug:

+            if (!string.IsNullOrEmpty(definition.DefaultFormula))
 +                field.DefaultValue = definition.DefaultFormula;

DefaultFormula must be used instead. Regression tests must be added to validate that particular property usage.

As per the community feedback:
https://www.yammer.com/spmeta2feedback/#/threads/show?threadId=737707933
@SubPointSupport SubPointSupport added this to the 2016-07-25 milestone Jul 19, 2016
@SubPointSupport SubPointSupport self-assigned this Jul 19, 2016
SubPointSupport added a commit that referenced this issue Jul 25, 2016
@SubPointSupport
+ SSOM - Fix DefaultFormula bug, add specific tests for DefaultFormul…
037bcd0 
…a property #855

+ failed and passed CanDeploy_Field_WithDefaultFormula test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SubPointSupport SubPointSupport

Labels
bug community-feedback contribution-easy
Projects
None yet
Milestone
2016-08-01
Development

No branches or pull requests
1 participant
@SubPointSupport