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

FormBuilderLocator with generic type can't always be found #977

Open
pbrant opened this issue Apr 17, 2011 · 4 comments
Open

FormBuilderLocator with generic type can't always be found #977

pbrant opened this issue Apr 17, 2011 · 4 comments
Assignees
Labels

Comments

@pbrant
Copy link
Member

pbrant commented Apr 17, 2011

manifest.toString() in vendForm() can return surprising results depending on the field(...) call in the screen.

Sample project here: https://github.com/pbrant/generic_fbl_test / git://github.com/pbrant/generic_fbl_test.git

The cause appears to be the generic valConversion() validation method

Original thread here http://groups.google.com/group/liftweb/browse_thread/thread/af21b61154c2b8c2#

@dpp
Copy link
Member

dpp commented Jul 1, 2011

Assigning the ticket to Peter now that he's a committer (bah hah hah... my evil plan is working. ;-) )

@pbrant
Copy link
Member Author

pbrant commented Jul 1, 2011

Aw, man.  :-)

I'll at least check to see if this has changed in 2.9.  I don't remember exactly how the manifest.toString()s differed, but it might also be possible to normalize the returned value to compare them reliably.  IIRC, it seemed like they were essentially saying the same thing in different ways.

@github-importer
Copy link

Imported from Assembla: http://www.assembla.com/spaces/liftweb/tickets/977

@ghost ghost assigned pbrant Mar 1, 2012
@Shadowfiend
Copy link
Member

Probably worth mentioning in here in case someone else wants to take up the gauntlet, in the original ML post David suggested that we look into directly comparing manifests instead of using toString. Not sure what latest behavior is on any of this though.

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

No branches or pull requests

4 participants