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

Export to Office 2007 #1765

Merged
merged 6 commits into from
Aug 25, 2016
Merged

Export to Office 2007 #1765

merged 6 commits into from
Aug 25, 2016

Conversation

Siedlerchr
Copy link
Member

@Siedlerchr Siedlerchr commented Aug 17, 2016

First fix for #1750.
I will continue reworking some other fields, too and will add some more tests.

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@@ -0,0 +1,15 @@
% Encoding: UTF-8

@InBook{Orlowski;2011,
Copy link
Member

Choose a reason for hiding this comment

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

Please name the test files after what they do and minimize them, not like the Stupro did just enumerating them 😄

@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Aug 23, 2016
@Siedlerchr
Copy link
Member Author

I would like to merge this in for 3,6 (No Localization changes), after I rebase

Remove empty line in changelog

Fix failing test and add new test file

Issue gets exported as issue
Journaltitle and journal both get exoprted as JournalName
patennumber gets exported correctly

Fix checkstyle

Create better structure for import test
TODO: Fix import of journal!

Importer maps to biblatex fields for most fields
JournalName maps to journaltitle
Month can be imported and exported
Patentnumber gets importerd as number when the entryType is a patent
Remove msbib-patentnumber
Correct tests, use utf8 encoding

Create new file for empty test to avoid conflict with other imports

Use Latex2Unicode formatter for Author names, too
Search/split "and" in authors field case insenstive

Some code cleanups
Use more optionals

Export number as issue, when issue field not present
Rework some test files

Removed some Globals, reworked Tests
Day is MSBib only
Make Language methods deprecated
@Siedlerchr Siedlerchr changed the title [WIP] Export to Office 2007 Export to Office 2007 Aug 23, 2016
@koppor koppor added this to the v3.6 milestone Aug 24, 2016

- Fixed [#1750](https://github.com/JabRef/jabref/issues/1750): BibLaTeX `date` field is now correctly exported as `year` in MS-Office 2007 xml format
- Fixed: LaTeX characters in author names are now converted to Unicode before export in MS-Office 2007 xml format
- Fixed: `Volume`, `Journaltitle`, `Issue` and `number`(for Patents) fields are now exported correctly in MS-Office 2007 xml format
Copy link
Member

Choose a reason for hiding this comment

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

all field names in lower case. Space after number. Patents with lower case "p"

@oscargus
Copy link
Contributor

Looks good! Just some minor comments.

@Siedlerchr
Copy link
Member Author

I will fix it this afternoon/evening. Regarding the language mapping, there is not yet a simple solution, as we would have to integrate the whole table of LCID:

https://msdn.microsoft.com/de-de/goglobal/bb964664.aspx
https://support.office.com/en-us/article/Locale-identification-numbers-for-language-specific-files-16102229-37c5-4ad5-92b1-0438f6304114?ui=en-US&rs=en-US&ad=US

@@ -77,12 +81,26 @@ public static BibEntry convert(MSBibEntry entry) {
fieldValues.put(MSBIB_PREFIX + "accessed", entry.dateAccessed);
}

if(entry.journalName != null)
{
Copy link
Member

Choose a reason for hiding this comment

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

{ in line above

* master:
  Some more Swedish translations (#1836)
  Updated Menu_tr.properties
  Updated jabref_tr.properties
  Add check for obsolete database structure. Add help popup. (#1818)
  Updated dependencies (#1831)
  Removed one Globals call from MetaData (#1782)
@Siedlerchr
Copy link
Member Author

Address comments as far as possible

* master:
  Disable incompatible properties for shared database. (#1826)
  Allow setting the etal-string empty. Implements #1841 (#1848)
  fix SplitPaneChangeListener (#1840)
  French localization: Jabref_fr: empty strings translated (#1847)
  Fixes #1687 "month" field ascending/descending sorting swapped (#1837)
  Fix localization entry.
  Update German translation (#1829)
  entry scrolling  didn't work if the entry in question was just 1 row outside the table (bottom) (<; <= mistake)
  fix indent
  when pasting an entry search the entry from the back, b/c we want to select the duplicate, not the original
  fix issues and delete a duplicate method
  when inserting a duplicate the right entry will be selected
@Siedlerchr Siedlerchr mentioned this pull request Aug 25, 2016
3 tasks
@Siedlerchr Siedlerchr merged commit d850ee4 into master Aug 25, 2016
@Siedlerchr Siedlerchr deleted the office07 branch August 25, 2016 12:33
ayanai1 pushed a commit to ayanai1/jabref that referenced this pull request Sep 5, 2016
* Export year to office 2007 correclty when only date is set in biblatex
Remove empty line in changelog
Fix failing test and add new test file
Issue gets exported as issue
Journaltitle and journal both get exoprted as JournalName
patennumber gets exported correctly
Create better structure for import test
Importer maps to biblatex fields for most fields
JournalName maps to journaltitle
Month can be imported and exported
Patentnumber gets importerd as number when the entryType is a patent
Remove msbib-patentnumber
Correct tests, use utf8 encoding
Create new file for empty test to avoid conflict with other imports
Use Latex2Unicode formatter for Author names, too
Search/split "and" in authors field case insenstive
Export number as issue, when issue field not present
Rework some test files
Day is MSBib only

* Rebased and addded new changelog entries

* Address comments, fix javadoc, changelog and add some StringBuilder
Remove some empty lines

* Merge changes and remove some empty lines
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