Skip to content

NHMA-35 - Automatic "access" attribute calculation #2

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Toshik
Copy link
Contributor

@Toshik Toshik commented Apr 14, 2015

Usually you should specify "access" attribute.
There are mapping errors when someone refactors code and converts field to/from property and forgets to change attribute value.

There is some automatisation:

  1. If you specify "access" explicit - it will be used
  2. If "access" is not specified - NHMA will determine if it is "field" or "property"

It works for all attribute types except ComponentAttribute, there you should specify "access" manually.

@Toshik Toshik changed the title log4net and nunit references path fix reference path fix and automatic "access" attribute value calculation Apr 14, 2015
@Toshik Toshik closed this Apr 14, 2015
@Toshik
Copy link
Contributor Author

Toshik commented Apr 14, 2015

I am not too familiar with pull requests. Last commit 7f560ea should not be in that request :)

@Toshik Toshik reopened this Apr 14, 2015
@rjperes

This comment was marked as outdated.

@Toshik

This comment was marked as outdated.

@Toshik

This comment was marked as outdated.

@rjperes

This comment was marked as outdated.

@Toshik
Copy link
Contributor Author

Toshik commented Apr 14, 2015

Ok, here it is: https://nhibernate.jira.com/browse/NHMA-35

Proper condition for "access" attribute auto generation
@hazzik hazzik changed the title reference path fix and automatic "access" attribute value calculation NHMA-35 - Automatic "access" attribute calculation May 19, 2017
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.

2 participants