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

Fixes #3505: Another try to fix the NPE in the search bar #3512

Merged
merged 1 commit into from
Dec 13, 2017
Merged

Conversation

tobiasdiez
Copy link
Member

@tobiasdiez tobiasdiez commented Dec 10, 2017

Fiexes #3505

I think, I managed to locate the problem of the NPE and the missing search bar. Apparently, in some conditions the syntax ../ to access the parent in a path does not work for resources. Thus the css file is not found, resulting in a NPE.


  • Change in CHANGELOG.md described (bug introduced in 4.1dev)
  • Tests created for changes
  • Screenshots added (for bigger UI changes)
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)
  • If you changed the localization: Did you run gradle localizationUpdate?

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Dec 10, 2017
Copy link
Member

@Siedlerchr Siedlerchr left a comment

Choose a reason for hiding this comment

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

Good catch. Seems really odd this error only occurs on certain systems

@koppor
Copy link
Member

koppor commented Dec 11, 2017

Search bar appears here. I think, @j0hannes should run the build at his system. Currently, one has to checkout the code and execute ./gradlew run as we currently have issues with our code signing certificate.

@j0hannes
Copy link

@koppor I just rebuilt the package from git and the search bar is still not there.

@tobiasdiez
Copy link
Member Author

@j0hannes did you checked-out the branch fix3505 or still used the master? Since this PR is not yet merged, the fix is not included in the master branch. If you used the correct branch to build, is the same error message still there?

@tobiasdiez
Copy link
Member Author

Anyway, I merge this now since the code still works under normal conditions and might help in others. So a new checkout of the master branch should work now.

@tobiasdiez tobiasdiez merged commit 47226fd into master Dec 13, 2017
@tobiasdiez tobiasdiez deleted the fix3505 branch December 13, 2017 00:02
@j0hannes
Copy link

@tobiasdiez Now it's back. The package will certainly only use the master branch; I though it was already merged there.

@tobiasdiez
Copy link
Member Author

@j0hannes thanks for the feedback and your help debugging this issue! Much appreciated.

Siedlerchr added a commit that referenced this pull request Dec 13, 2017
* upstream/master: (108 commits)
  Fetcher for IACR eprints (#3473)
  Update internal state of DatabaseChangeMonitor when external changes … (#3503)
  Fixes #3505: Another try to fix the NPE in the search bar (#3512)
  Replace ' with ' so that our HTML preview can handle it correctly
  Added a "Clear text" button in right click menu within the text boxes. (#3475)
  Add reset to English language after a test
  New translations JabRef_en.properties (German)
  Remove ampersand in non-menu localizations
  New translations JabRef_en.properties (German)
  New translations Menu_en.properties (German)
  New translations Menu_en.properties (German)
  New translations JabRef_en.properties (Vietnamese)
  New translations JabRef_en.properties (Italian)
  New translations Menu_en.properties (Italian)
  New translations JabRef_en.properties (Indonesian)
  New translations Menu_en.properties (Indonesian)
  New translations JabRef_en.properties (Greek)
  New translations Menu_en.properties (Greek)
  New translations Menu_en.properties (Japanese)
  New translations JabRef_en.properties (German)
  ...

# Conflicts:
#	build.gradle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants