Goml doug logic util #4890

dbruening53 · 2019-04-15T11:42:20Z

Change in CHANGELOG.md described
Tests created for changes
Manually tested changed features in running JabRef
Screenshots added in PR description (for bigger UI changes)
Ensured that the git commit message is a good one
Check documentation status (Issue created for outdated help page at help.jabref.org?)

src\test\java\org\jabref\CatchExceptionsFromThread.java

asdfa

Google coding standard for line length is 100 characters. This fix breaks one long line of 103 characters down into 3 smaller lines. No logic was changed.

need functional TestFX for more testing

…operly.

…g-BugBibTexKeyDuplicateCheckthrowsNPE GOML JB Week5

…itDistanceIgnoreCaseRefactor Refactor to meet line length standard

…g-BugBibTexKeyDuplicateCheckthrowsNPE Goml jb duplicate resolver dialog bug bib tex key duplicate checkthrows npe

…pWindowScale changed the Edit Group dialog window to show full window on screen pr…

Goml doug refactor aux parser

Created test class and 2 test cases on the get() method

…tAndGetFunctions Made a test class testing the RftCharMap java class

Refactor: regex null test deleted

Add test cases for StringUtil.getPart()

GOML JB Week6

Goml hieu casechanger

tested: constuctor & getEncoding methods

added one line comment in ImportFormatPreferencesTest

standards

# Conflicts: # src/test/java/org/jabref/model/strings/StringUtilTest.java

…oug-LogicUtil

tobiasdiez · 2019-04-15T18:45:38Z

Thanks for your pull-request. Since there are quite a few changes, can you please explain a bit what you did, i.e. what is fixed or changed. Thanks!

LinusDietz · 2019-04-24T13:42:39Z

Hey @dbruening53, @jabesse @dinhh2 @BlueWokou @benjagooder!
Thanks for your combined efforts in improving JabRef.

Can you please merge the current master into this branch to let the Travis tests run?
Can you (as @tobiasdiez asked) make some statement on what is happening here?
Like this, we have no way of moving forward with this PR, which would be a shame, since you already put so much effort into this!

dbruening53 · 2019-04-24T19:42:24Z

My sincere apologies. I had responded to an earlier email regarding this merge request. We forked JabRef as a school project and mostly added some unit test cases. I intended to merge into our fork and mistakenly merged into the primary branch. Please ignore this merge request. Thanks! Doug Bruening and the rest of the "Git Off My Lawn" team

…

________________________________ From: Linus Dietz <[email protected]> Sent: Wednesday, April 24, 2019 9:42 AM To: JabRef/jabref Cc: Douglas Bruening; Mention Subject: Re: [JabRef/jabref] Goml doug logic util (#4890) Hey @dbruening53<https://github.com/dbruening53>, @jabesse<https://github.com/jabesse> @dinhh2<https://github.com/dinhh2> @BlueWokou<https://github.com/BlueWokou> @benjagooder<https://github.com/benjagooder>! Thanks for your combined efforts in improving JabRef. * Can you please merge the current master into this branch to let the Travis tests run? * Can you (as @tobiasdiez<https://github.com/tobiasdiez> asked) make some statement on what is happening here? Like this, we have no way of moving forward with this PR, which would be a shame, since you already put so much effort into this! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#4890 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AGBPODSVY6767NSYMDIXEKLPSBPVJANCNFSM4HF7CI5Q>.

Siedlerchr · 2019-04-24T20:13:05Z

Okay, I closed this PR. However, if you are able to spot or fix a bug based on the latest master using your unit tests you are of course welcome to create a PR here at our repo.

a995c63 Create errata.csl (JabRef#4894) 883b9cd Update archeologie-medievale.csl (JabRef#4892) a942b5c Update rmit-university-harvard.csl (JabRef#4890) 5d535fc Update universidade-do-porto-faculdade-de-engenharia-chicago.csl (JabRef#4891) f2a8fe3 Fix AVMA style c5a2d7e Create universidade-do-porto-faculdade-de-engenharia-chicago.csl (JabRef#4872) 9f106eb Update rmit-university-harvard.csl (JabRef#4884) 5213c2f Update universitetet-i-oslo-rettsvitenskap.csl (JabRef#4886) 7e0828d Update uclouvain-centre-charles-de-visscher-pour-le-droit-internation… (JabRef#4871) ae5eba3 Create revue-archeologique-de-lest.csl (JabRef#4881) 58a1672 Create pacific-conservation-biology.csl (JabRef#4880) 381106e Update national-institute-of-organisation-dynamics-australia-harvard.csl (JabRef#4879) c66a5c0 Create society-of-biblical-literature-author-date.csl (JabRef#4812) 008073c Create ruhr-universitat-bochum-medizinische-fakultat-variante-a.csl (JabRef#4874) 193385a Update manchester (JabRef#4878) fa9ef11 Create Faculty of Psychology and Education Sciences, Porto (JabRef#4856) 2f70b14 Add "personal communication" to remaining APA locales (JabRef#4866) 2b4f831 Fixed page-range-format (JabRef#4863) d5661ba universitat-bern-institut-fur-musikwissenschaft add missing cs:sort to cs:bibliography (JabRef#4860) e433c99 fix delimiter issue for bioscience.csl (JabRef#4876) 4d97aab update et-al settings for stroke.csl (JabRef#4875) 342b9e3 Update masarykova-univerzita-pravnicka-fakulta.csl (JabRef#4868) 91fe25e Add article number to IEEE (JabRef#4867) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: a995c63

a995c63 Create errata.csl (#4894) 883b9cd Update archeologie-medievale.csl (#4892) a942b5c Update rmit-university-harvard.csl (#4890) 5d535fc Update universidade-do-porto-faculdade-de-engenharia-chicago.csl (#4891) f2a8fe3 Fix AVMA style c5a2d7e Create universidade-do-porto-faculdade-de-engenharia-chicago.csl (#4872) 9f106eb Update rmit-university-harvard.csl (#4884) 5213c2f Update universitetet-i-oslo-rettsvitenskap.csl (#4886) 7e0828d Update uclouvain-centre-charles-de-visscher-pour-le-droit-internation… (#4871) ae5eba3 Create revue-archeologique-de-lest.csl (#4881) 58a1672 Create pacific-conservation-biology.csl (#4880) 381106e Update national-institute-of-organisation-dynamics-australia-harvard.csl (#4879) c66a5c0 Create society-of-biblical-literature-author-date.csl (#4812) 008073c Create ruhr-universitat-bochum-medizinische-fakultat-variante-a.csl (#4874) 193385a Update manchester (#4878) fa9ef11 Create Faculty of Psychology and Education Sciences, Porto (#4856) 2f70b14 Add "personal communication" to remaining APA locales (#4866) 2b4f831 Fixed page-range-format (#4863) d5661ba universitat-bern-institut-fur-musikwissenschaft add missing cs:sort to cs:bibliography (#4860) e433c99 fix delimiter issue for bioscience.csl (#4876) 4d97aab update et-al settings for stroke.csl (#4875) 342b9e3 Update masarykova-univerzita-pravnicka-fakulta.csl (#4868) 91fe25e Add article number to IEEE (#4867) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: a995c63

a995c63 Create errata.csl (JabRef#4894) 883b9cd Update archeologie-medievale.csl (JabRef#4892) a942b5c Update rmit-university-harvard.csl (JabRef#4890) 5d535fc Update universidade-do-porto-faculdade-de-engenharia-chicago.csl (JabRef#4891) f2a8fe3 Fix AVMA style c5a2d7e Create universidade-do-porto-faculdade-de-engenharia-chicago.csl (JabRef#4872) 9f106eb Update rmit-university-harvard.csl (JabRef#4884) 5213c2f Update universitetet-i-oslo-rettsvitenskap.csl (JabRef#4886) 7e0828d Update uclouvain-centre-charles-de-visscher-pour-le-droit-internation… (JabRef#4871) ae5eba3 Create revue-archeologique-de-lest.csl (JabRef#4881) 58a1672 Create pacific-conservation-biology.csl (JabRef#4880) 381106e Update national-institute-of-organisation-dynamics-australia-harvard.csl (JabRef#4879) c66a5c0 Create society-of-biblical-literature-author-date.csl (JabRef#4812) 008073c Create ruhr-universitat-bochum-medizinische-fakultat-variante-a.csl (JabRef#4874) 193385a Update manchester (JabRef#4878) fa9ef11 Create Faculty of Psychology and Education Sciences, Porto (JabRef#4856) 2f70b14 Add "personal communication" to remaining APA locales (JabRef#4866) 2b4f831 Fixed page-range-format (JabRef#4863) d5661ba universitat-bern-institut-fur-musikwissenschaft add missing cs:sort to cs:bibliography (JabRef#4860) e433c99 fix delimiter issue for bioscience.csl (JabRef#4876) 4d97aab update et-al settings for stroke.csl (JabRef#4875) 342b9e3 Update masarykova-univerzita-pravnicka-fakulta.csl (JabRef#4868) 91fe25e Add article number to IEEE (JabRef#4867) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: a995c63

* Squashed 'src/main/resources/csl-styles/' changes from 586e0b8..c5f14e2 c5f14e2 Update journal-of-plant-ecology.csl (#4837) b5d00ab Hotfix: pravnik.csl & iso690-full-note-cs.csl (#4833) fb8d48a add PMID and DOI (#4832) 7ebdfff Update lancaster-university-harvard.csl (#4836) 8771e21 Create journal-of-clinical-neurology.csl (#4835) 70beb7b Harvard Lancaster: Fix in-text locator 0709b73 Update informal-logic.csl (#4831) d18d133 Update gost-r-7-0-5-2008.csl (#4829) cfb5389 Update gost-r-7-0-5-2008-numeric.csl (#4828) 0fc46ea Update gost-r-7-0-5-2008-numeric-alphabetical.csl (#4827) fac26cb Create feminist-theory.csl (#4830) a2721e6 Update pravnik.csl, masarykova-univerzita-pravnicka-fakulta.csl, and iso690-full-note-cs.csl (#4826) 2f0d735 Improve localization of DGfP 683d7d1 Create zeitschrift-fur-theologie-und-philosophie.csl (#4813) 7c82406 always show DOI in aps.csl (#4820) 1eb531d Update journal-of-international-business-studies.csl (#4819) 9caf926 Create interdisziplinare-anthropologie.csl (#4818) 00210e1 UKSW2 (#4807) 015db6e UKSW1 (#4800) f706ef5 Update masarykova-univerzita-pravnicka-fakulta.csl (#4822) 096108d Update epilepsia.csl (#4823) 07ebdc3 Update eksploatacja-i-niezawodnosc.csl (#4824) 3d50f3c Create ABNT-Universidade-Federal-de-Pernambuco-CFCH.csl (#4798) df94fa1 Update eksploatacja-i-niezawodnosc.csl (#4808) 4089be5 Update bulletin-de-la-societe-prehistorique-francaise.csl (#4810) f130aa7 Update molecular-biology-and-evolution.csl (#4814) 86f452f Create heiliger-dienst.csl (#4815) 5ccd3e9 Update revista-brasileira-de-ciencia-do-solo.csl (#4817) 0eea734 Fix a delimiter in journal-of-the-royal-society-of-western-australia.csl (#4811) 1cbe790 Delete heiliger-dienst.csl 19fa29f Create heiliger-dienst.csl (#4787) 888bad1 Update RMIT Harvard to match Easy Cite (#4788) 46011dd Create universite-nangui-abrogoua-ufr-sn (#4676) 174adab Create apa-6th-edition-no-ampersand.csl (#4767) 03cf65f Create journal-of-plant-protection-research.csl (#4797) c749084 Update eksploatacja-i-niezawodnosc.csl (#4782) 17e1501 Update masarykova-univerzita-pravnicka-fakulta.csl (#4783) 5596d18 Update revista-brasileira-de-ciencia-do-solo.csl (#4790) 9c1a381 create csl for the journal Textual Cultures (#4785) 8345762 Update taylor-and-francis-national-library-of-medicine.csl (#4786) 5ec1acf Create british-journal-of-criminology.csl (#4789) 68550ac Update angewandte-chemie.csl (#4791) 09e2755 Create shock.csl (#4792) d46e49e Update antioxidants-and-redox-signaling.csl (#4793) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: c5f14e2 * Squashed 'src/main/resources/csl-styles/' changes from c5f14e2..716f635 716f635 Update changes in Harvard-Stellenbosch (#4862) a6a9a1a Create alkoholizmus-a-drogove-zavislosti.csl (#4859) 0560ca0 Update journal-of-the-american-college-of-surgeons.csl (#4858) 9e16631 create independent style for cellular-and-molecular-gastroenterology-and-hepatology.csl (#4852) 87b0853 Update antiquity.csl (#4841) cb4e7f7 Update heart-rhythm.csl (#4850) b3b55ee Create iisue-moderno (#4846) 63973a7 Create etasr.csl (#4848) 9942138 Create style for Journal of Vertebrate Biology (#4849) 7c7f3d3 Add Future Medicine dependents (#4838) abe2066 Update methods-of-information-in-medicine.csl (#4847) c64d98b create independent style for new-zealand-journal-of-forestry-science (#4844) 28265f9 Create continuity-and-change.csl (#4839) d065fdd Create journal-of-clinical-ethics.csl (#4842) b9fbfa3 Update future-medicine.csl (#4843) ceb56fa Create future-medicine.csl (#4834) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: 716f635 * Squashed 'src/main/resources/csl-styles/' changes from 716f635..a995c63 a995c63 Create errata.csl (#4894) 883b9cd Update archeologie-medievale.csl (#4892) a942b5c Update rmit-university-harvard.csl (#4890) 5d535fc Update universidade-do-porto-faculdade-de-engenharia-chicago.csl (#4891) f2a8fe3 Fix AVMA style c5a2d7e Create universidade-do-porto-faculdade-de-engenharia-chicago.csl (#4872) 9f106eb Update rmit-university-harvard.csl (#4884) 5213c2f Update universitetet-i-oslo-rettsvitenskap.csl (#4886) 7e0828d Update uclouvain-centre-charles-de-visscher-pour-le-droit-internation… (#4871) ae5eba3 Create revue-archeologique-de-lest.csl (#4881) 58a1672 Create pacific-conservation-biology.csl (#4880) 381106e Update national-institute-of-organisation-dynamics-australia-harvard.csl (#4879) c66a5c0 Create society-of-biblical-literature-author-date.csl (#4812) 008073c Create ruhr-universitat-bochum-medizinische-fakultat-variante-a.csl (#4874) 193385a Update manchester (#4878) fa9ef11 Create Faculty of Psychology and Education Sciences, Porto (#4856) 2f70b14 Add "personal communication" to remaining APA locales (#4866) 2b4f831 Fixed page-range-format (#4863) d5661ba universitat-bern-institut-fur-musikwissenschaft add missing cs:sort to cs:bibliography (#4860) e433c99 fix delimiter issue for bioscience.csl (#4876) 4d97aab update et-al settings for stroke.csl (#4875) 342b9e3 Update masarykova-univerzita-pravnicka-fakulta.csl (#4868) 91fe25e Add article number to IEEE (#4867) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: a995c63 * Squashed 'src/main/resources/csl-styles/' changes from a995c63..bf698ac bf698ac Create common-market-law-review.csl (#4910) c962eca Create harvard-prifysgol-caerdydd.csl (#4922) 0c24e7f Update gewerblicher-rechtsschutz-und-urheberrecht.csl (#4923) d2ec1a7 Create Tijdschrift-voor-Geneeskunde.csl (#4907) 5df7250 Update harvard-institut-fur-praxisforschung-de.csl (#4918) 093fd91 Update universite-de-montreal-apa.csl (#4916) a3e41d4 Update thieme-german.csl (#4919) 648765a add DOI to aerosol-science-and-technology.csl (#4909) bc1ebee Reindent/reorder a8dc18a Fix documentation link for epidemiology & infection aab403a Fix AGLC Newspaper date 4c018d5 Add period between editor and translator (SBL styles) (#4906) 42f7491 Create geographische-zeitschrift.csl (#4898) a4002a6 Create german-kunstwissenschaft.csl (#4896) b01910b Disambiguation of names (#4895) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: bf698ac * Squashed 'src/main/resources/csl-styles/' changes from bf698ac..7cab2f7 7cab2f7 Create depro-ufs.csl (#4947) 2fcda1f Create journal-of-sport-science-and-medicine.csl (#4949) cd457d4 Update american-marketing-association.csl (#4945) cacc4ee Create nejm-catalyst.csl (#4943) bd769b9 Update and rename dependent/chinese-medical-journal.csl to chinese-me… (#4941) 1f706cd Update health-services-research.csl (#4939) 76bcd1d Update journal-of-the-royal-society-of-western-australia.csl (#4932) e79640e Create afro-asia.csl (#4934) c601aa4 Update lancaster-university-harvard.csl (#4938) 184fd90 Update collection-du-centre-jean-berard.csl (#4936) 82f9aec fix et-al & add DOI thyroid.csl (#4937) 4af169f Create universidade-estadual-de-alagoas-uneal-abnt (#4885) 47165b3 Update and rename medical-physics.csl to dependent/medical-physics.csl (AMA) (#4905) 3cab27d Update thieme-german.csl (#4931) eb2c977 Create zeitschrift-fur-zahnarztliche-implantologie.csl (#4925) 4adb1ea Create karstenia.csl (#4929) 530a136 Fix author substitute in Universita Cattolica git-subtree-dir: src/main/resources/csl-styles git-subtree-split: 7cab2f7 * Squashed 'src/main/resources/csl-styles/' changes from 7cab2f7..827b986 827b986 add DOI preprint to american-society-for-microbiology.csl (#4946) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: 827b986 * Squashed 'src/main/resources/csl-styles/' changes from 827b986..eb0d37e eb0d37e Create natbib-plainnat-author-dat.csl (#4967) fb1592a Update journal-of-fish-biology.csl (#4969) f6876cb Update .travis.yml (#4970) d6d400b Create london-review-of-international-law.csl (#4966) d2a5ae1 Update harvard-stellenbosch-university.csl (#4965) 9c62141 Create phytopathologia-mediterranea.csl (#4964) 8ca2ea1 Update historical-materialism.csl (#4960) 80456dc Update historical-materialism.csl be2d910 Update historical-materialism.csl 1d1cf09 Create atlande.csl (#4930) 89f41d4 Create juristische-zitierweise-oeffentliches-recht.csl (#4944) 8c677a0 Create korean-journal-of-gastroenterology.csl (#4954) eadb950 Create historical-materialism.csl (#4955) 5553dcd Create revista-materia.csl (#4957) d23a3ab Bug fix in APA 6 original publication macro (#4959) 66f9974 Always print publisher in APA 6th edition (#4899) 868809c Create agora.csl (#4940) 99c19c3 Update anthropologie-et-societes.csl (#4952) ee17423 Create critical-reviews-in-solid-state-and-materials-science.csl (#4951) 7a13a7d Create korean-journal-of-internal-medicine.csl (#4953) git-subtree-dir: src/main/resources/csl-styles git-subtree-split: eb0d37e * Merged new library actions * l10n * Removed unused constructor * Fixed checkstyle failed check * Fixed one more checkstyle issue Co-authored-by: github actions <[email protected]>

benjagooder and others added 30 commits February 7, 2019 17:50

Update to trivial comment

0906993

src\test\java\org\jabref\CatchExceptionsFromThread.java

Grammar update to trivial comment

954a8c4

Made a new test class

41e011f

asdfa

move to GOML week5 branch

3c1e1f6

Refactor to meet line length standard

13478cc

Google coding standard for line length is 100 characters. This fix breaks one long line of 103 characters down into 3 smaller lines. No logic was changed.

Tested as much as possible without JavaFX unit tests;

1636f78

need functional TestFX for more testing

changed the Edit Group dialog window to show full window on screen pr…

df4f85e

…operly.

Merge pull request #3 from benjagooder/GOML-JB-DuplicateResolverDialo…

2e62b01

…g-BugBibTexKeyDuplicateCheckthrowsNPE GOML JB Week5

Merge pull request #4 from benjagooder/GOML-Benja-StringSimilarity-ed…

b2536bd

…itDistanceIgnoreCaseRefactor Refactor to meet line length standard

Merge pull request #5 from benjagooder/GOML-JB-DuplicateResolverDialo…

b2ae219

…g-BugBibTexKeyDuplicateCheckthrowsNPE Goml jb duplicate resolver dialog bug bib tex key duplicate checkthrows npe

Merge branch 'GOML-Hieu-GroupDialog-EditGroupWindowScale' into GOML

6370793

Merge pull request #6 from benjagooder/GOML-Hieu-GroupDialog-EditGrou…

8c4da44

…pWindowScale changed the Edit Group dialog window to show full window on screen pr…

Update AuxParserTest.java

a888d55

Merge pull request #7 from benjagooder/GOML_Doug_refactorAuxParser

479ffcd

Goml doug refactor aux parser

Update DuplicateResolverDialogTest.java

f30dde5

Merge remote-tracking branch 'origin/GOML' into GOML

11335e1

Made a test class testing the RftCharMap java class

94e1422

Created test class and 2 test cases on the get() method

Merge pull request #8 from benjagooder/GOML-Benja-RTFCharMap-tetingPu…

192f6cc

…tAndGetFunctions Made a test class testing the RftCharMap java class

Add test cases for StringUtil.getPart()

e4d438b

Create MoveFieldCleanupTest.java

43569bf

Merge branch 'BuddyUnitTest' into GOML

71d3711

Added a lot of get tests for logic.formatter.bibtexfields

523f2b1

Refactor: regex null test deleted

Merge branch 'GOML' into GOML-JB-Week6

c651421

Merge pull request #9 from benjagooder/GOML-Doug-StringUtilTest

e31c1cd

Add test cases for StringUtil.getPart()

Merge pull request #10 from benjagooder/GOML-JB-Week6

2042f8d

GOML JB Week6

Merge pull request #11 from benjagooder/GOML_Hieu_casechanger

d850b63

Goml hieu casechanger

Create ImportFormatPreferencesTest.java

d7358d7

Started ImportFormatPreferencesTest class

7b4c39a

tested: constuctor & getEncoding methods

first commit on rebase test branch

8191870

added one line comment in ImportFormatPreferencesTest

second commit on rebase test branch

5e8473d

added one line comment in ImportFormatPreferencesTest

Hieu and others added 7 commits March 16, 2019 16:24

Edited AutosaveManagerTest.java and MetaDataTest.java to fit CheckStyle

8b4a3c2

standards

Merge remote-tracking branch 'origin/GOML-Dev' into GOML-Dev

712312f

# Conflicts: # src/test/java/org/jabref/model/strings/StringUtilTest.java

unit tests for Password class in logic.shared.security branch

75df87b

unit test for Password class in logic.shared.security

fa7ed4a

Create SharedDatabasePreferencesTest.java

b6e4f39

Merge remote-tracking branch 'origin/GOML-Doug-LogicUtil' into GOML-D…

15cfdac

…oug-LogicUtil

Add test coverage for logic.util.UpdateFieldPreferences

8432b45

LinusDietz added the status: waiting-for-feedback The submitter or other users need to provide more information about the issue label Apr 24, 2019

Siedlerchr closed this Apr 24, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Goml doug logic util #4890

Goml doug logic util #4890

dbruening53 commented Apr 15, 2019

tobiasdiez commented Apr 15, 2019

LinusDietz commented Apr 24, 2019

dbruening53 commented Apr 24, 2019 via email

Siedlerchr commented Apr 24, 2019

Goml doug logic util #4890

Goml doug logic util #4890

Conversation

dbruening53 commented Apr 15, 2019

tobiasdiez commented Apr 15, 2019

LinusDietz commented Apr 24, 2019

dbruening53 commented Apr 24, 2019 via email

Siedlerchr commented Apr 24, 2019