Skip to content

Commit

Permalink
JabRef#8491 : Fix Check Style errors pt 2
Browse files Browse the repository at this point in the history
  • Loading branch information
addak committed Feb 26, 2022
1 parent 98e71b9 commit 24f2aa3
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 13 deletions.
13 changes: 9 additions & 4 deletions src/main/java/org/jabref/logic/xmp/DublinCoreExtractor.java
Original file line number Diff line number Diff line change
Expand Up @@ -419,7 +419,9 @@ private void fillTitle(String title) {
*
* @param coverage
*/
private void fillCoverage(String coverage) { dcSchema.setCoverage(coverage); }
private void fillCoverage(String coverage) {
dcSchema.setCoverage(coverage);
}

/**
* BibTex Field : language ; DC Field : dc:language
Expand All @@ -443,17 +445,19 @@ private void fillLanguages(String languages) {

/**
* BibTex : Rights (Custom Field); DC Field : dc:rights
*
* @param rights
*/
private void fillRights(String rights){
private void fillRights(String rights) {
dcSchema.addRights(null, rights.split(",")[0]);
}

/**
* BibTex : Source (Custom Field); DC Field : Source
*
* @param source
*/
private void fillSource(String source){
private void fillSource(String source) {
dcSchema.setSource(source);
}

Expand Down Expand Up @@ -513,8 +517,9 @@ public void fillDublinCoreSchema() {
break;
case MONTH:
case DAY:
if (hasStandardYearField)
if (hasStandardYearField) {
break;
}
default:
this.fillCustomField(field.getKey(), field.getValue());
}
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/org/jabref/logic/xmp/XmpUtilReader.java
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@

import org.jabref.model.entry.BibEntry;
import org.jabref.model.entry.LinkedFile;
import org.jabref.model.schema.DublinCoreSchemaCustom;

import org.apache.pdfbox.Loader;
import org.apache.pdfbox.pdmodel.PDDocument;
import org.apache.pdfbox.pdmodel.PDDocumentCatalog;
import org.apache.pdfbox.pdmodel.PDDocumentInformation;
import org.apache.pdfbox.pdmodel.common.PDMetadata;
import org.apache.xmpbox.XMPMetadata;
import org.jabref.model.schema.DublinCoreSchemaCustom;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

Expand Down
2 changes: 1 addition & 1 deletion src/main/java/org/jabref/logic/xmp/XmpUtilWriter.java
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
import org.jabref.model.entry.BibEntry;
import org.jabref.model.entry.field.Field;
import org.jabref.model.entry.field.StandardField;
import org.jabref.model.schema.DublinCoreSchemaCustom;

import org.apache.pdfbox.Loader;
import org.apache.pdfbox.pdmodel.PDDocument;
Expand All @@ -28,7 +29,6 @@
import org.apache.xmpbox.XMPMetadata;
import org.apache.xmpbox.schema.DublinCoreSchema;
import org.apache.xmpbox.xml.XmpSerializer;
import org.jabref.model.schema.DublinCoreSchemaCustom;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
package org.jabref.model.schema;

import java.util.Objects;
import javax.xml.XMLConstants;

import org.apache.commons.lang3.exception.ExceptionUtils;
import org.apache.commons.lang3.reflect.FieldUtils;
import org.apache.xmpbox.XMPMetadata;
Expand All @@ -11,9 +14,6 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import javax.xml.XMLConstants;
import java.util.Objects;

@StructuredType(preferedPrefix = "dc", namespace = "http://purl.org/dc/elements/1.1/")
public class DublinCoreSchemaCustom extends DublinCoreSchema {

Expand Down
10 changes: 6 additions & 4 deletions src/test/java/org/jabref/logic/xmp/XmpUtilWriterTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,9 @@
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.io.TempDir;

import static org.jabref.logic.xmp.DublinCoreExtractor.*;
import static org.jabref.logic.xmp.DublinCoreExtractor.DC_COVERAGE;
import static org.jabref.logic.xmp.DublinCoreExtractor.DC_RIGHTS;
import static org.jabref.logic.xmp.DublinCoreExtractor.DC_SOURCE;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
Expand Down Expand Up @@ -71,10 +73,10 @@ private void initBibEntries() {
vapnik2000.setField(StandardField.AUTHOR, "Vladimir N. Vapnik");
vapnik2000.setField(StandardField.DOI, "10.1007/978-1-4757-3264-1");
vapnik2000.setField(StandardField.OWNER, "Ich");
vapnik2000.setField(StandardField.LANGUAGE,"English, Japanese");
vapnik2000.setDate(new Date(2314,11,1));
vapnik2000.setField(StandardField.LANGUAGE, "English, Japanese");
vapnik2000.setDate(new Date(2314, 11, 1));

vapnik2000.setField(new UnknownField(DC_COVERAGE),"coverageField");
vapnik2000.setField(new UnknownField(DC_COVERAGE), "coverageField");
vapnik2000.setField(new UnknownField((DC_SOURCE)), "JabRef");
vapnik2000.setField(new UnknownField(DC_RIGHTS), "Right To X");
}
Expand Down

0 comments on commit 24f2aa3

Please sign in to comment.