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

Adding docs about null parameters #663

Merged
merged 2 commits into from
Jul 4, 2014
Merged

Adding docs about null parameters #663

merged 2 commits into from
Jul 4, 2014

Conversation

Turini
Copy link
Member

@Turini Turini commented Jul 4, 2014

related with #532 and #660

@garcia-jj
Copy link
Member

The default should "null when not present" if I remember. Or not?

@garcia-jj
Copy link
Member

Looking the code, the default behavior is null when no parameter was found. https://github.com/caelum/vraptor4/pull/660/files#diff-ac6ae891d48b200fe6c56de7d8dc126bR98

@Turini
Copy link
Member Author

Turini commented Jul 4, 2014

ops.. I've changed the method condition but forgot to push 8a1ba5a The default
behavior of VRaptor 4 is always instantiate method parameters, I didn't change it.

@garcia-jj
Copy link
Member

Ok. 🔫

Turini added a commit that referenced this pull request Jul 4, 2014
Adding docs about null parameters
@Turini Turini merged commit 29dbacf into master Jul 4, 2014
@Turini Turini deleted the addingDocsFor532PR branch July 4, 2014 19:46
@Turini
Copy link
Member Author

Turini commented Jul 4, 2014

thanks :)

@Turini
Copy link
Member Author

Turini commented Jul 17, 2014

Hi Cácio, there is a release coming soon, probably next monday.

Rodrigo Turini
Caelum | Ensino e Inovação
www.caelum.com.br

2014-07-16 17:51 GMT-03:00 Cácio Costa notifications@github.com:

Hi, do you know when this feature will be released? I'm working on JBoss
Forge plugin for VRaptor 4 to agilize (scaffold) development in my
organization (Supremo Tribunal Federal), and the generated @manytoone
attributes throws org.hibernate.TransientPropertyValueException because it
contains an parent Entity with null identifier.


Reply to this email directly or view it on GitHub
#663 (comment).

@garcia-jj
Copy link
Member

Por favor, poste dúvidas em nossa mailing list. Este local é para
discussões sobre as issues.

@lucascs
Copy link
Member

lucascs commented Jul 18, 2014

Caio, abra uma issue ou um Pull Request mudando esse método pra protected

On Thu, Jul 17, 2014 at 11:03 AM, Otávio Garcia notifications@github.com
wrote:

Por favor, poste dúvidas em nossa mailing list. Este local é para
discussões sobre as issues.


Reply to this email directly or view it on GitHub
#663 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants