Skip to content

#8 Don't use autowired or constructor injection with FactoryBeans #12

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

Conversation

frommeyerc
Copy link

Changes in line with the JavaDoc on org.springframework.beans.factory.FactoryBean<T> use the BeanFactory instead of constructor autowired beans by implementing BeanFactoryAware.

@codecov-io
Copy link

codecov-io commented Jan 28, 2020

Codecov Report

Merging #12 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop      #12      +/-   ##
=============================================
+ Coverage      67.05%   67.07%   +0.01%     
- Complexity       663      664       +1     
=============================================
  Files             84       84              
  Lines           2289     2290       +1     
  Branches         402      402              
=============================================
+ Hits            1535     1536       +1     
  Misses           570      570              
  Partials         184      184
Impacted Files Coverage Δ Complexity Δ
...amodb/repository/config/DynamoDBMapperFactory.java 100% <100%> (ø) 4 <3> (+1) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 435ad90...0fe4197. Read the comment docs.

@boostchicken boostchicken merged commit be44208 into boostchicken:develop Feb 2, 2020
boostchicken added a commit that referenced this pull request Feb 25, 2020
* Read Consistency Setting

* #8

Custom Mapper Spec

* #8

Custom Mapper CDI fix

* Fix symlink to CONTRIBUTING.md (#13)

* #8 Don't use autowired or constructor injection with FactoryBeans (#12)

* [maven-release-plugin] prepare release v5.2.2

* [maven-release-plugin] prepare for next development iteration

Co-authored-by: Christian Frommeyer <[email protected]>
boostchicken added a commit that referenced this pull request Feb 25, 2020
* Read Consistency Setting

* #8

Custom Mapper Spec

* #8

Custom Mapper CDI fix

* Fix symlink to CONTRIBUTING.md (#13)

* #8 Don't use autowired or constructor injection with FactoryBeans (#12)

* [maven-release-plugin] prepare release v5.2.2

* [maven-release-plugin] prepare for next development iteration

* 5.2.2 docs fix

* Hibernate Validator CVE fix
GHSA-m8p2-495h-ccmh

* Documentation updates

* Create FUNDING.yml

* Fixed a Type (#17)

Co-authored-by: thedevluffy <[email protected]>

* Documentation updates

* 5.2.3 prep

* [maven-release-plugin] prepare release v5.2.3

* [maven-release-plugin] prepare for next development iteration

Co-authored-by: Christian Frommeyer <[email protected]>
Co-authored-by: thedevluffy <[email protected]>
boostchicken added a commit that referenced this pull request Jun 17, 2020
* v5.2.3 (#18)

* Read Consistency Setting

* #8

Custom Mapper Spec

* #8

Custom Mapper CDI fix

* Fix symlink to CONTRIBUTING.md (#13)

* #8 Don't use autowired or constructor injection with FactoryBeans (#12)

* [maven-release-plugin] prepare release v5.2.2

* [maven-release-plugin] prepare for next development iteration

* 5.2.2 docs fix

* Hibernate Validator CVE fix
GHSA-m8p2-495h-ccmh

* Documentation updates

* Create FUNDING.yml

* Fixed a Type (#17)

Co-authored-by: thedevluffy <[email protected]>

* Documentation updates

* 5.2.3 prep

* [maven-release-plugin] prepare release v5.2.3

* [maven-release-plugin] prepare for next development iteration

Co-authored-by: Christian Frommeyer <[email protected]>
Co-authored-by: thedevluffy <[email protected]>

* ✨ Allow CONTAINS and NOT_CONTAINS is repositories

* 🎨 cleanup

* 🔧 check for collection

* 🔧 check for collection

* ✏️ cleanup

* 🎨 cleanup imports

* Remove funding changes

Co-authored-by: John D <[email protected]>
Co-authored-by: Christian Frommeyer <[email protected]>
Co-authored-by: thedevluffy <[email protected]>
Co-authored-by: John Dorman <[email protected]>
Co-authored-by: Hannes Angst <[email protected]>
boostchicken added a commit that referenced this pull request Jun 17, 2020
* Update FUNDING.yml

* Introduce string set handling (#34)

* v5.2.3 (#18)

* Read Consistency Setting

* #8

Custom Mapper Spec

* #8

Custom Mapper CDI fix

* Fix symlink to CONTRIBUTING.md (#13)

* #8 Don't use autowired or constructor injection with FactoryBeans (#12)

* [maven-release-plugin] prepare release v5.2.2

* [maven-release-plugin] prepare for next development iteration

* 5.2.2 docs fix

* Hibernate Validator CVE fix
GHSA-m8p2-495h-ccmh

* Documentation updates

* Create FUNDING.yml

* Fixed a Type (#17)

Co-authored-by: thedevluffy <[email protected]>

* Documentation updates

* 5.2.3 prep

* [maven-release-plugin] prepare release v5.2.3

* [maven-release-plugin] prepare for next development iteration

Co-authored-by: Christian Frommeyer <[email protected]>
Co-authored-by: thedevluffy <[email protected]>

* ✨ Allow CONTAINS and NOT_CONTAINS is repositories

* 🎨 cleanup

* 🔧 check for collection

* 🔧 check for collection

* ✏️ cleanup

* 🎨 cleanup imports

* Remove funding changes

Co-authored-by: John D <[email protected]>
Co-authored-by: Christian Frommeyer <[email protected]>
Co-authored-by: thedevluffy <[email protected]>
Co-authored-by: John Dorman <[email protected]>
Co-authored-by: Hannes Angst <[email protected]>

* Updating changelog for #33

* Updating for release

* [maven-release-plugin] prepare release v5.2.5

* [maven-release-plugin] prepare for next development iteration

Co-authored-by: Hannes Angst <[email protected]>
Co-authored-by: Christian Frommeyer <[email protected]>
Co-authored-by: thedevluffy <[email protected]>
Co-authored-by: Hannes Angst <[email protected]>
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.

4 participants