Now it's responsible for count if items in group (enable / disable, based on user preferences)

In case we can't use PreferencesService injection, I propose to create a property (displayItemsCount) and set this property from outside (from GroupTreeView) as True/False. True - need get count, False - NO need to get count of items.
Also I found method "onDatabaseChanged" - it fires when you delete/add new items to group. So we need "displayItemsCount" to restrict calculation in this case also.

By default displayItemsCount = False

…ter is active

…abref into fix-for-issue-5891

My method (calculateNumberOfMatches) is no longer valid and GroupNodeViewModel must use a new methods called "findMatches"

otherwise it will be interpreted as path which obiously fails

Fixes JabRef#6507

Return true in action helper if file is online link

* Added validators and some visual changes to ExternalTab

* Added ExternalApplicationsPreferences

* Refactored for reduced complexity

* Refactored for eye-candy

Fix fetcher help links

Extend the bib file for better screenshots

# Conflicts:
#	src/main/java/org/jabref/preferences/JabRefPreferences.java
#	src/main/java/org/jabref/preferences/PreferencesService.java

Bumps [flexmark-ext-gfm-tasklist](https://github.com/vsch/flexmark-java) from 0.61.30 to 0.61.32.
- [Release notes](https://github.com/vsch/flexmark-java/releases)
- [Commits](https://github.com/vsch/flexmark-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [flexmark](https://github.com/vsch/flexmark-java) from 0.61.30 to 0.61.32.
- [Release notes](https://github.com/vsch/flexmark-java/releases)
- [Commits](https://github.com/vsch/flexmark-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [archunit-junit5-api](https://github.com/TNG/ArchUnit) from 0.13.1 to 0.14.1.
- [Release notes](https://github.com/TNG/ArchUnit/releases)
- [Commits](TNG/ArchUnit@v0.13.1...v0.14.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

…sch.flexmark-flexmark-ext-gfm-tasklist-0.61.32

Bump flexmark-ext-gfm-tasklist from 0.61.30 to 0.61.32

…sch.flexmark-flexmark-0.61.32

Bump flexmark from 0.61.30 to 0.61.32

Bumps [flexmark-ext-gfm-strikethrough](https://github.com/vsch/flexmark-java) from 0.61.30 to 0.61.32.
- [Release notes](https://github.com/vsch/flexmark-java/releases)
- [Commits](https://github.com/vsch/flexmark-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

…sch.flexmark-flexmark-ext-gfm-strikethrough-0.61.32

Bump flexmark-ext-gfm-strikethrough from 0.61.30 to 0.61.32

…ech.archunit-archunit-junit5-api-0.14.1

Bump archunit-junit5-api from 0.13.1 to 0.14.1

Fixed an issue where a new entry is not shown in the library if a filter/search is active

…-update

Add some javadoc and fix the preview update issue

…6508)

Allow entrytype as sorting criterion

…#6504)

…r-issue-5891

)

* Fix Pattern.compile for frequently used regexes

* Fix one additional Pattern.compile

* Fix style and unnecessary escape sequences

* Fix invalid index in call to substring

The original condition is evaluated to false.
The substring is shorter than "uni".

* Refactor name and javadoc of a regex

* Fix use of compiled regex for matching department

* Fix check for uppercase letter

Perhaps the assumption should be that the letters are ASCII.
If all letters are ASCII checking 'A' <= k.charAt(0) <= 'Z'
might make more sense.
I am not convinced about doing this with a regex.

* Fix usage of uncompiled regex

* Fix readability?

* Add test cases

Both test cases involves an author name containing department or school
 without university or institute of technology.

* Fix `null` appearing as part of author name

Corporate authors without university/institute of technology

* Refactor name of capital regex pattern

* Add debug output for reordering of names in fields

* Add helper methods

* Fix missing negation in "uni" matching

* Fix test cases for corporate authors

* Fix to keep all uppercase letters in abbreviation

* Fix commented out code

* Fix key for institution's name containing keyword

If the name of an institution can't be split, assume that
"School"/"Department" is part of the name.

* Fix test case for short institution name

* Refactor check for institution types

* Refactor comments and names improving readability?

* Refactor to improve readability and closure

* Fix JavaDoc

Minor typos and the "rest" part is now created differently.

* Fix JavaDoc typos

* Fix preliminary order for authors -> latexfree

* Drop logger

* Add convenience methods for cached latexfree names

* Add name format method for names containing latex

* Add call to formatNameLatexFree

* Fix unclear statement in JavaDoc

* Fix to only keep the first character of each word

There are some examples that will turn out wrong if only capital letters
 are kept, e.g., "iOS Developer University Program".
 The original problem with "The School of Life" becoming too short is
 avoided by only removing school/department for names containing two or
  more ',' separated strings. This will still produce an unexpected
   result if the address of the institution is part of the authors field

* Add latexfree Natbib test cases

* Fix typo in latex-free test cases

* Add Natbib test with escaped brackets

* Add Natbib institution test with escaped brackets

* Add test for latex-free comma separated lastnames

* Add test for latex-free comma separated first name

First name first and abbreviated first name first

* Add test for latex-free comma separated last name

Last name first and abbreviated first names

* Fix adherence to JavaDoc and readability(?)

* Fix readability(?)

* Fix CheckStyle issues

The deprecated static methods BibtexKeyGenerator.generateKey are moved
to the test file as a similar convenience method is required for
the test cases. Suppress warning has been added for some methods.

* Fix CHANGELOG.md

* Fix mistake in BibtexKeyGeneratorTest

generateKey was not copy-pasted properly.

* Add test for oxford comma

* Fix miss-capitalization of enum

* Fix fields not displayed latex-free

* Fix in-line methods in MainTableNameFormatter

* Fix in-line of generateKey() method

* Fix separating tests into parsing/representation

* Fix cache check and simplify expressions

* Drop inlined methods

* Fix most abbreviated abbreviations

* Drop old formatName method

* Refactor formatNameLatexFree

The author list parsing is moved outside of the if/else statements

* Refactor new parse tests

* Add more parse tests

* Drop all test cases containing escaped brackets

* Refactor parse with latex tests

Move them close to other parse tests

* Fix my own spelling mistakes

* Refactor abbreviation name

# Conflicts:
#	src/main/java/org/jabref/gui/maintable/BibEntryTableViewModel.java
#	src/main/java/org/jabref/gui/maintable/MainTableNameFormatter.java

Removed BIB file directory from search when preferences has option unchecked

Refactor main table preferences

Bumps [flexmark](https://github.com/vsch/flexmark-java) from 0.61.32 to 0.62.0.
- [Release notes](https://github.com/vsch/flexmark-java/releases)
- [Commits](https://github.com/vsch/flexmark-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [flexmark-ext-gfm-tasklist](https://github.com/vsch/flexmark-java) from 0.61.32 to 0.62.0.
- [Release notes](https://github.com/vsch/flexmark-java/releases)
- [Commits](https://github.com/vsch/flexmark-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [bcprov-jdk15on](https://github.com/bcgit/bc-java) from 1.65 to 1.65.01.
- [Release notes](https://github.com/bcgit/bc-java/releases)
- [Changelog](https://github.com/bcgit/bc-java/blob/master/docs/releasenotes.html)
- [Commits](https://github.com/bcgit/bc-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

…cycastle-bcprov-jdk15on-1.65.01

Bump bcprov-jdk15on from 1.65 to 1.65.01

…sch.flexmark-flexmark-0.62.0

Bump flexmark from 0.61.32 to 0.62.0

…sch.flexmark-flexmark-ext-gfm-tasklist-0.62.0

Bump flexmark-ext-gfm-tasklist from 0.61.32 to 0.62.0

Bumps [flexmark-ext-gfm-strikethrough](https://github.com/vsch/flexmark-java) from 0.61.32 to 0.62.0.
- [Release notes](https://github.com/vsch/flexmark-java/releases)
- [Commits](https://github.com/vsch/flexmark-java/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>

…8-6042(2)

Now it's responsible for count if items in group (enable / disable, based on user preferences)

In case we can't use PreferencesService injection, I propose to create a property (displayItemsCount) and set this property from outside (from GroupTreeView) as True/False. True - need get count, False - NO need to get count of items.
Also I found method "onDatabaseChanged" - it fires when you delete/add new items to group. So we need "displayItemsCount" to restrict calculation in this case also.

By default displayItemsCount = False