Skip to content

Conversation

@JarredOlson
Copy link

I emailed you the other day about this issue. I looked into the code and for some reason the asType method on SearchControls.groovy is not getting invoked. I tried several things including changing the signature to public Object asType(Class c) {...} directly invoking the asType() method. I tried to recreate this in a test project and could not. My fix is not ideal but it seems to get the job done. I couldn't get the tests to run cleanly on my local machine (before and after making changes I had 12 failures) so could you please let me know if you run into any issues when building?

@ldaley
Copy link
Owner

ldaley commented Jan 9, 2012

Many thanks for this. I'll get it in very soon.

On 5 Jan 2012, at 18:43, Jarred Olson wrote:

I emailed you the other day about this issue. I looked into the code
and for some reason the asType method on SearchControls.groovy is not
getting invoked. I tried several things including changing the
signature to public Object asType(Class c) {...} directly invoking the
asType() method. I tried to recreate this in a test project and could
not. My fix is not ideal but it seems to get the job done. I
couldn't get the tests to run cleanly on my local machine (before and
after making changes I had 12 failures) so could you please let me
know if you run into any issues when building?

You can merge this Pull Request by running:

git pull https://github.com/JarredOlson/Gldapo master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • Updating the Gldapo Search functionality to work with Grails 2.0 &
    groovy 1.8

-- File Changes --

M src/main/groovy/gldapo/GldapoDirectory.groovy (2)
M src/main/groovy/gldapo/search/SearchControls.groovy (18)

-- Patch Links --

https://github.com/alkemist/Gldapo/pull/1.patch
https://github.com/alkemist/Gldapo/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

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