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

#2772 issue fixed #3552

Closed
wants to merge 6 commits into from
Closed

#2772 issue fixed #3552

wants to merge 6 commits into from

Conversation

PJozeph
Copy link
Contributor

@PJozeph PJozeph commented Dec 19, 2017

Fixes #2772


  • Change in CHANGELOG.md described
  • 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?

@PJozeph PJozeph closed this Dec 19, 2017
@PJozeph PJozeph reopened this Dec 19, 2017
Copy link
Member

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

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

Thanks, this looks like a good start. However, it seems that you missed to change the string to FileExtensions in a few places. Eg, the build fails because of

/home/travis/build/JabRef/jabref/src/main/java/org/jabref/preferences/CustomExportList.java:93: error: incompatible types: String cannot be converted to FileExtensions

ExportFormat format = new ExportFormat(s.get(0), s.get(0), lfFileName, null, s.get(2), layoutPreferences,

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 19, 2017

okay, sorry i started travis again, i will fix it and will come back

@Siedlerchr
Copy link
Member

Good start, you should also take a look at the export dialog where the export formats are used to see if it can be changed there, too

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 20, 2017

Changeed the string to FileExtensions at all places.

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 20, 2017

How can i check in Travis where i broke the code ?

@Siedlerchr
Copy link
Member

@PJozeph Select one of the travis jobsm e.g. modernizer and click on it it out:
Seems like you forgot some pieces of code. Eclipse should have already warned you about build errors,
But the error log on travis looks odd. I restarted the job
https://travis-ci.org/JabRef/jabref/jobs/319061181

@@ -20,22 +20,7 @@
import javax.xml.datatype.XMLGregorianCalendar;
import javax.xml.namespace.QName;

import org.jabref.logic.importer.fileformat.bibtexml.Article;
Copy link
Member

Choose a reason for hiding this comment

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

The problem are these removed imports (and similarly further down)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i dont have org.jabref.logic.importer.fileformat.bibtexml folder

@LinusDietz
Copy link
Member

There is still a problem with the imports. try running ./gradlew check on your local computer to avoid the delay through travis

@Siedlerchr
Copy link
Member

In Eclipse you can run CTRL+Shift+O to organize your imports

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 20, 2017

that's right,but since the project does not have org.jabref.logic.importer.fileformat.bibtexml folder the CTRL+Shift+O will remove those imports @Siedlerchr

@Siedlerchr
Copy link
Member

Then you have messed up sth. I will take a look at your branch

@Siedlerchr
Copy link
Member

Ah okay, you have an import name clash. If you simply remove the first import of java.io.File it should work again. (I tested it locally)

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 20, 2017

I dont understand this. The ModsExportFormat and BibTeXMLExportFormat with full of compiler error there are many Classes that cannot be resolved. I still dont have org.jabref.logic.importer.fileformat.bibtexml package.

Removed the first java.io.File in the ModsExportFormat and BibTeXMLExportFormat class but got the same issue.

What i see after running ./gradlew check is that the build fails due to this compiler errors.

jabref2
t the same issue.
jabref

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 20, 2017

jabrefgradlewcheck

@Siedlerchr
Copy link
Member

I created a patch file which fixes the imports and your build:
0004-fix-imports.patch.txt

To import it, remove the .txt extension and do:
git am < 0004-fix-imports.patch

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 20, 2017

unfortunately , no it does not

jabrefpatch

@tobiasdiez
Copy link
Member

tobiasdiez commented Dec 20, 2017

This really looks like a problem with your setup. The code in question (bibtexml/...) is automatically generated if I'm not mistaken. Can you try to run ./gradlew cleanEclipse, ./gradlew eclipse and ./gradlew generateSource?

@PJozeph
Copy link
Contributor Author

PJozeph commented Dec 21, 2017

i decided to rebuild the working environment. But the build failed and i got a exception as : Execution failed for task ':junitPlatformTest'.

Process 'command '/usr/lib/jvm/java-8-oracle/bin/java'' finished with non-zero exit value 1

@PJozeph PJozeph closed this Dec 21, 2017
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