Skip to content

Feature/add abstract field to complex search#7079

Merged
calixtus merged 3 commits into
JabRef:masterfrom
DominikVoigt:feature/add-abstract-field-to-complex-search
Nov 7, 2020
Merged

Feature/add abstract field to complex search#7079
calixtus merged 3 commits into
JabRef:masterfrom
DominikVoigt:feature/add-abstract-field-to-complex-search

Conversation

@DominikVoigt

Copy link
Copy Markdown
Contributor

This PR adds the abstract field as a fielded term for complex queries and implements it for the existing complex search fetchers that support abstract field based filtering.

  • Change in CHANGELOG.md described (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

Signed-off-by: Dominik Voigt <dominik.ingo.voigt@gmail.com>

@koppor koppor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small nitpick comment

Comment thread src/main/java/org/jabref/logic/importer/fetcher/ComplexSearchQuery.java Outdated
Comment thread src/main/java/org/jabref/logic/importer/fetcher/ComplexSearchQuery.java Outdated
Signed-off-by: Dominik Voigt <dominik.ingo.voigt@gmail.com>

@calixtus calixtus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@calixtus

calixtus commented Nov 7, 2020

Copy link
Copy Markdown
Member

Two approvals -> I think I'll merge this.
Thanks for the work. I hope you succeeded in your exams well.

@calixtus calixtus merged commit df80c13 into JabRef:master Nov 7, 2020
Siedlerchr added a commit to JabRef/jabref-koppor that referenced this pull request Nov 11, 2020
* upstream/master: (164 commits)
  Bump lucene-queryparser from 8.6.3 to 8.7.0 (JabRef#7089)
  Fix endnote importer when keyword style is null (JabRef#7084)
  Rename Firefox extension file in Windows (JabRef#7092)
  Add support for Microsoft Edge browser in Windows and Linux builds (JabRef#7056)
  Bump unirest-java from 3.11.03 to 3.11.05 (JabRef#7087)
  Bump com.github.ben-manes.versions from 0.33.0 to 0.36.0 (JabRef#7088)
  Bump gittools/actions from v0.9.4 to v0.9.5 (JabRef#7091)
  Feature/add abstract field to complex search (JabRef#7079)
  Add missing authors
  Implement Emacs key bindings (JabRef#6037)
  Special field code maintenance (JabRef#7078)
  Follow up fix for 7077 Reset preview layouts on clear
  Fix preview settings not saved due to l10n (JabRef#7077)
  Add link to existing documentation of filed types
  Remove unused supportsPaging
  Squashed 'src/main/resources/csl-styles/' changes from 5c376b8..f4399aa
  Fix more 404 links
  Fix custom theme styles not applied to the entry preview (JabRef#7071)
  Fix Shared Database Tests (JabRef#7040)
  Bump byte-buddy-parent from 1.10.17 to 1.10.18 (JabRef#7059)
  ...

# Conflicts:
#	src/main/java/org/jabref/gui/DefaultInjector.java
@DominikVoigt DominikVoigt deleted the feature/add-abstract-field-to-complex-search branch February 11, 2021 15:35
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.

3 participants