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

Issue54 #56

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

Issue54 #56

wants to merge 4 commits into from

Conversation

ianrae
Copy link

@ianrae ianrae commented Mar 14, 2015

Hi @hakandilek. Here are two projects: twixt54 is a play module,and m8 is a play app.

Steps
-build and publish-local twixt54
-build m8
Both these projects have play2-crud as a dependency

The key line is in m8 TaxiController.java.

public class TaxiController extends MyDynamicTwixtController<Long, Taxi, TaxiTwixt>

When the controller is derived from the local class MyDynamicTwixtController, everything works fine and the
ClasspathScanningControllerRegistry sees TaxiController:

[debug] p.u.m.c.ClasspathScanningControllerRegistry - pqpcontrollerClass : class
controllers.TaxiController

But when TaxiController is changed to derive from DynamicTwixtController (in twixt54), then ClasspathScanningControllerRegistry does not see TaxiController. I am no longer seeing an exception, but play2-crud simply ignores TaxiController and generates its own. This is incorrect; it should see that TaxiController is a crud controller for the Taxi model.

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.

1 participant