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

Prefix for JobScope changed to scopedTarget #4353

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MrCoffee77
Copy link

@MrCoffee77 MrCoffee77 commented Apr 14, 2023

I had a problem with JobScope when xml and java-config is mixed.

I could solve it, with changing the TARGET_NAME_PREFIX in the class JobScope.
I also wrote a JUnit-Test for this fix.

@fmbenhassine
Copy link
Contributor

Thank you for your PR!

I had a problem with JobScope when xml and java-config is mixed.

Can you please elaborate on the problem? Is this similar to #3938?

@fmbenhassine fmbenhassine added the status: waiting-for-reporter Issues for which we are waiting for feedback from the reporter label May 23, 2023
@MrCoffee77
Copy link
Author

yes it is similar, I used for the PR the same solution. In this #3938 farnetto also suggested it.

I hope it helps you to fix this issue.

@fmbenhassine
Copy link
Contributor

OK thank you for your feedback. I am just trying to understand the problem this PR is trying to solve to document it in the change log. I will try the test you added in this PR without the fix to see the issue.

@fmbenhassine fmbenhassine added status: feedback-provided Issues for which the feedback requested from the reporter was provided pr-for: bug in: core related-to: batch-scopes and removed status: waiting-for-reporter Issues for which we are waiting for feedback from the reporter labels May 23, 2023
@MrCoffee77
Copy link
Author

Yes, with the Test you will see, that the Framework builds a second Proxy around the JobScoped-Bean, with other interfaces

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core pr-for: bug related-to: batch-scopes status: feedback-provided Issues for which the feedback requested from the reporter was provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants