[Bug 59830] "Skipped only -1 while trying to skip 67116544 bytes. This should never happen." IOException in VBAMacroReader

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59830

bro...@usinternet.com changed:

   What|Removed |Added

  Attachment #34027|macro_virus.doc |macro_virus.doc.do_not_open
   filename||
  Attachment #34027|Example doc that causes |MACRO VIRUS INFECTED WORD
description|readMacros() to throw an|DOC, DO NOT OPEN IN WORD!
   |IOException |It causes readMacros() to
   ||throw an IOException

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59833] Promote FillPatternType-based getFillPattern/setFillPattern to Common SS

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59833

Javen O'Neal  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 OS|Linux   |All
 Status|NEW |RESOLVED

--- Comment #1 from Javen O'Neal  ---
Applied in r1751972.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59833] New: Promote FillPatternType-based getFillPattern/setFillPattern to Common SS

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59833

Bug ID: 59833
   Summary: Promote FillPatternType-based
getFillPattern/setFillPattern to Common SS
   Product: POI
   Version: 3.15-dev
  Hardware: PC
OS: Linux
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: SS Common
  Assignee: dev@poi.apache.org
  Reporter: one...@apache.org

FillPatternType enum was added in r701797 to XSSFCellStyle, but never added to
HSSFCellStyle. In the interest of making the HSSF and XSSF share the same
Common SS API and in the interest of having typed parameters, HSSFCellStyle
should have the same changes.

We should favor the FillPatternType enum variants and in the future remove the
non-typed methods that operate on shorts.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 55555] Adding a border to a cell with background pattern sets the pattern color to black

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=5

--- Comment #16 from Javen O'Neal  ---
Applied pull request from comment 14 in r1751955 with corresponding unit test.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[GitHub] poi pull request #33: Bug 55555 fix

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/poi/pull/33


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 55555] Adding a border to a cell with background pattern sets the pattern color to black

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=5

--- Comment #15 from Qualtagh  ---
I'm sorry. It fixes only similar HSSF issue. Just a half of a problem.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59740] [PATCH] Sheet.shiftRows incorrectly shifts merged region on exists merged region

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59740

Javen O'Neal  changed:

   What|Removed |Added

 Status|NEW |NEEDINFO

--- Comment #4 from Javen O'Neal  ---
(In reply to Danil Lopatin from comment #3)
> If you want, I could update documentation.

Yes, please include a patch for the updated documentation, and I will commit
the code changes and documentation together.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56154] [CoreProperties] - get|set - lastModifiedBy missing

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56154

--- Comment #6 from Javen O'Neal  ---
I renamed these new methods to getLastModifiedByUser and setLastModifiedByUser
for clarity.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56154] [CoreProperties] - get|set - lastModifiedBy missing

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56154

Javen O'Neal  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #5 from Javen O'Neal  ---
Applied in r1751948, r1751949, and r1751950. Will be included in POI 3.15 beta
3 release.

Updated changelog in r1751952.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59830] "Skipped only -1 while trying to skip 67116544 bytes. This should never happen." IOException in VBAMacroReader

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59830

bro...@usinternet.com changed:

   What|Removed |Added

 CC||bro...@usinternet.com

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59830] New: "Skipped only -1 while trying to skip 67116544 bytes. This should never happen." IOException in VBAMacroReader

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59830

Bug ID: 59830
   Summary: "Skipped only -1 while trying to skip 67116544 bytes.
This should never happen." IOException in
VBAMacroReader
   Product: POI
   Version: 3.15-dev
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: POIFS
  Assignee: dev@poi.apache.org
  Reporter: bro...@usinternet.com

Created attachment 34027
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=34027=edit
Example doc that causes readMacros() to throw an IOException

WARNING: ATTACHED DOCUMENT IS INFECTED WITH A MALICIOUS MACRO. DO NOT OPEN IT
USING MICROSOFT WORD.

I have been using POI to help take apart Office documents and look for
suspicious/malicious content. I had developed my own way to extract the VBA
script in the embedded macros, but have found the new VBAMacroReader to be much
more useful.

However, I have noticed that VBAMacroReader fails on certain Office documents.
I have attached one such example.


Code used to reproduce the error:

File file = new File("macro_virus.doc");
VBAMacroReader reader = new VBAMacroReader(file);
Map macros = reader.readMacros();


Stack trace:
java.io.IOException: Skipped only -1 while trying to skip 67116544 bytes.  This
should never happen.
at
org.apache.poi.poifs.macros.VBAMacroReader.trySkip(VBAMacroReader.java:182)
at
org.apache.poi.poifs.macros.VBAMacroReader.readMacros(VBAMacroReader.java:240)
at
org.apache.poi.poifs.macros.VBAMacroReader.findMacros(VBAMacroReader.java:148)
at
org.apache.poi.poifs.macros.VBAMacroReader.findMacros(VBAMacroReader.java:153)
at
org.apache.poi.poifs.macros.VBAMacroReader.findMacros(VBAMacroReader.java:153)
at
org.apache.poi.poifs.macros.VBAMacroReader.readMacros(VBAMacroReader.java:115)
at poitester.POITester.main(POITester.java:27)


I've successfully opened the file in a sandbox and it appears to be a valid
Word document.

Also, just FYI, I am using poi-3.15-beta2 and I get the same results on OSX
10.10.5 and openSUSE 11.4. 

Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56154] [CoreProperties] - get|set - lastModifiedBy missing

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56154

Danil Lopatin  changed:

   What|Removed |Added

  Attachment #34026|0   |1
   is patch||

--- Comment #4 from Danil Lopatin  ---
Comment on attachment 34026
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=34026
moved tests to separate method

>Index: src/ooxml/java/org/apache/poi/POIXMLProperties.java
>===
>--- src/ooxml/java/org/apache/poi/POIXMLProperties.java(revision 
>1751911)
>+++ src/ooxml/java/org/apache/poi/POIXMLProperties.java(working copy)
>@@ -301,6 +301,12 @@
> public void setLastPrinted(String date) {
> part.setLastPrintedProperty(date);
> }
>+public String getLastModifiedBy() {
>+return part.getLastModifiedByProperty().getValue();
>+}
>+public void setLastModifiedBy(String lastModifiedBy) {
>+part.setLastModifiedByProperty(lastModifiedBy);
>+}
> public Date getModified() {
> return part.getModifiedProperty().getValue();
> }
>Index: src/ooxml/testcases/org/apache/poi/TestPOIXMLProperties.java
>===
>--- src/ooxml/testcases/org/apache/poi/TestPOIXMLProperties.java   
>(revision 1751911)
>+++ src/ooxml/testcases/org/apache/poi/TestPOIXMLProperties.java   
>(working copy)
>@@ -192,7 +192,7 @@
> 
> XWPFDocument doc2 = XWPFTestDataSamples.writeOutAndReadBack(doc);
> doc.close();
>-cp = doc.getProperties().getCoreProperties();
>+cp = doc2.getProperties().getCoreProperties();
> Date dt3 = cp.getCreated();
> assertEquals(dateCreated, dt3);
> doc2.close();
>@@ -208,6 +208,14 @@
> assertEquals("20", _coreProperties.getRevision());
> }
> 
>+@Test
>+public void testLastModifiedByProperty() {
>+String lastModifiedBy = _coreProperties.getLastModifiedBy();
>+assertEquals("Paolo Mottadelli", lastModifiedBy);
>+_coreProperties.setLastModifiedBy("Test User");
>+assertEquals("Test User", _coreProperties.getLastModifiedBy());
>+}
>+
> public static boolean dateTimeEqualToUTCString(Date dateTime, String 
> utcString) {
> Calendar utcCalendar = 
> LocaleUtil.getLocaleCalendar(LocaleUtil.TIMEZONE_UTC);
> utcCalendar.setTimeInMillis(dateTime.getTime());

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56154] [CoreProperties] - get|set - lastModifiedBy missing

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56154

Danil Lopatin  changed:

   What|Removed |Added

  Attachment #34025|0   |1
is obsolete||

--- Comment #3 from Danil Lopatin  ---
Created attachment 34026
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=34026=edit
moved tests to separate method

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 55555] Adding a border to a cell with background pattern sets the pattern color to black

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=5

Qualtagh  changed:

   What|Removed |Added

 Status|NEEDINFO|NEW

--- Comment #14 from Qualtagh  ---
Added a pull request on GitHub with a fix of this bug:
https://github.com/apache/poi/pull/33

I have tested this issue after applying this fix using a java file attached by
Gilles. It solves the problem.

Fix description:
CellUtil was setting background color BEFORE setting foreground and fill
pattern. And HSSFCellStyle.setFillBackgroundColor javadoc says that background
color should be set AFTER setting fill pattern and foreground.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[GitHub] poi pull request #33: Bug 55555 fix

[Qualtagh]

GitHub user Qualtagh opened a pull request:

https://github.com/apache/poi/pull/33

Bug 5 fix

This is a fix for [bug 
5](https://bz.apache.org/bugzilla/show_bug.cgi?id=5) (background color 
is reset to black after calls to CellUtil).

Javadoc for `HSSFCellStyle.setFillBackgroundColor` says "Note: Ensure 
Foreground color is set prior to background". So, the order of operations in 
`CellUtil.setFormatProperties` should be different. It's ordered 
lexicographically now:

```java
style.setFillBackgroundColor(getShort(properties, FILL_BACKGROUND_COLOR));
style.setFillForegroundColor(getShort(properties, FILL_FOREGROUND_COLOR));
style.setFillPattern(getShort(properties, FILL_PATTERN));
```

According to HSSFCellStyle's javadoc, it should be so:

```java
style.setFillPattern(getShort(properties, FILL_PATTERN));
style.setFillForegroundColor(getShort(properties, FILL_FOREGROUND_COLOR));
style.setFillBackgroundColor(getShort(properties, FILL_BACKGROUND_COLOR));
```

Also, bugs [50849](https://bz.apache.org/bugzilla/show_bug.cgi?id=50849) 
and [59442](https://bz.apache.org/bugzilla/show_bug.cgi?id=59442) _may_ have 
the same cause (I didn't check).

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Qualtagh/poi trunk

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/poi/pull/33.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #33


commit b6ccc6e68fea84b47376ecab9281d55c73c58456
Author: Qualtagh 
Date:   2016-07-08T15:20:23Z

The order of operations should be different as stated in 
HSSFCellStyle.setFillBackgroundColor.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56154] [CoreProperties] - get|set - lastModifiedBy missing

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56154

Danil Lopatin  changed:

   What|Removed |Added

 Status|NEEDINFO|NEW

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56154] [CoreProperties] - get|set - lastModifiedBy missing

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56154

Danil Lopatin  changed:

   What|Removed |Added

 CC||solid.da...@gmail.com

--- Comment #2 from Danil Lopatin  ---
Created attachment 34025
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=34025=edit
patch and small test

Please apply this patch to 3.15 version.
It simply adds setter and getter for lastModifiedBy.
Test with core properties has been updated.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59740] [PATCH] Sheet.shiftRows incorrectly shifts merged region on exists merged region

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59740

--- Comment #3 from Danil Lopatin  ---
I hope this fix will be included in 3.15 version.
If you want, I could update documentation.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Jenkins build is back to normal : POI #1435

[Apache Jenkins Server]

See 


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Jenkins build is back to normal : POI-JDK8 #1125

[Apache Jenkins Server]

See 


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

RE: DrawingML

[Murphy, Mark]

My reading of the specs tells me that there are three common "parts" handled by 
DrawingML. Those are Charts, Diagrams, and Themes. The only real difference 
between the three project areas (Spreadsheet, document, presentation) vs. 
DrawingML is the way the part in question is referenced in the package. This 
could be handled in the createChart, createDiagram, or createTheme methods the 
specific package.

-Original Message-
From: Nick Burch [mailto:apa...@gagravarr.org] 
Sent: Friday, July 08, 2016 8:09 AM
To: POI Developers List 
Subject: Re: DrawingML

On Fri, 8 Jul 2016, Murphy, Mark wrote:
> Much of the Chart code and other things are common parts between 
> Spreadsheets, Documents, and Presentations. At least for the XML side 
> of things. I propose that the DrawingML portions of these three 
> projects be consolidated into a separate package or group of packages 
> to deal with things like charts and themes. If it is done correctly, 
> this code should be able to be shared among all areas of POI (XML 
> format anyway). I haven't checked into the older binary formats, but I 
> suspect that there could be a similar relationship.

For the binary formats, we have the "DDF" package which contains the common 
drawing records and classes. Sounds like we might be able to create a XDDF 
package for the common DrawingML drawing xml stuff too!

Nick


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, 
e-mail: dev-h...@poi.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Re: DrawingML

[Nick Burch]

On Fri, 8 Jul 2016, Murphy, Mark wrote:
Much of the Chart code and other things are common parts between 
Spreadsheets, Documents, and Presentations. At least for the XML side of 
things. I propose that the DrawingML portions of these three projects be 
consolidated into a separate package or group of packages to deal with 
things like charts and themes. If it is done correctly, this code should 
be able to be shared among all areas of POI (XML format anyway). I 
haven't checked into the older binary formats, but I suspect that there 
could be a similar relationship.


For the binary formats, we have the "DDF" package which contains the 
common drawing records and classes. Sounds like we might be able to create 
a XDDF package for the common DrawingML drawing xml stuff too!


Nick


-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Re: DrawingML

[Javen O'Neal]

Is DrawingML the OOXML counterpart to the Escher classes (DDF -
Dreadful Drawing Format)?

On Fri, Jul 8, 2016 at 5:00 AM, Murphy, Mark  wrote:
> Much of the Chart code and other things are common parts between 
> Spreadsheets, Documents, and Presentations. At least for the XML side of 
> things. I propose that the DrawingML portions of these three projects be 
> consolidated into a separate package or group of packages to deal with things 
> like charts and themes. If it is done correctly, this code should be able to 
> be shared among all areas of POI (XML format anyway). I haven't checked into 
> the older binary formats, but I suspect that there could be a similar 
> relationship.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

DrawingML

[Murphy, Mark]

Much of the Chart code and other things are common parts between Spreadsheets, 
Documents, and Presentations. At least for the XML side of things. I propose 
that the DrawingML portions of these three projects be consolidated into a 
separate package or group of packages to deal with things like charts and 
themes. If it is done correctly, this code should be able to be shared among 
all areas of POI (XML format anyway). I haven't checked into the older binary 
formats, but I suspect that there could be a similar relationship.

[Bug 59625] Localisation (Internationalisation in other languages) when applied in charts corrupt the MS Word file.

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59625

Mark Murphy  changed:

   What|Removed |Added

 Status|NEW |NEEDINFO

--- Comment #1 from Mark Murphy  ---
Please attach the Word document as well. There are differences between how Word
and Excel use the chart part. Particularly in how the chart data is referenced.
Look at the Open Office XML 1st edition Part 1: Fundamentals, December 2006 p.
125 lines 8-17.

I can't tell what the issue is without the word document. I suspect that you
are generating much of the XML yourself using the CT classes since the Word API
(and charts) is incomplete.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59826] Apply ant-generated patch.tar.gz patches

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59826

Javen O'Neal  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #1 from Javen O'Neal  ---
Added in r1751883.

Copy patch.tar.gz file to svn working copy root (the patch tarball must be
named patch.tar.gz and it must be located in the root)
Usage: ant -f patch.xml apply

Basically, what the script does is:

$ tar xvzf patch.tar.gz
$ svn patch patch.txt
$ tar --list -f patch.tar.gz | grep --invert-match patch\.txt | xargs svn add

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

FW: 685A4D4CD403EC0

[poi-dev]

685A4D4CD403EC0.docm
Description: application/vnd.ms-word.document.macroenabled.12

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Build failed in Jenkins: POI #1434

[Apache Jenkins Server]

See 

Changes:

[onealj] add crude unit test for ZipPackagePropertiesMarshaller

[onealj] convert to junit4

[onealj] whitespace and +svn:eol-style=native

--
Started by an SCM change
[EnvInject] - Loading node environment variables.
Building remotely on ubuntu-6 (docker Ubuntu ubuntu yahoo-not-h2) in workspace 

Updating http://svn.apache.org/repos/asf/poi/trunk at revision 
'2016-07-08T10:34:42.591 +'
U 
test-data/openxml4j/OPCCompliance_CoreProperties_AlternateTimezones.docx
U test-data/spreadsheet/StructuredReferences.xlsx
U src/scratchpad/testcases/org/apache/poi/hmef/TestHMEFMessage.java
U 
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFreeformShape.java
U src/scratchpad/src/org/apache/poi/hmef/HMEFMessage.java
U 
src/scratchpad/src/org/apache/poi/hmef/extractor/HMEFContentsExtractor.java
U src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
U 
src/ooxml/testcases/org/apache/poi/ss/formula/TestStructuredReferences.java
U 
src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestPackageCoreProperties.java
D 
src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/AllOpenXML4JInternalTests.java
AU
src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/TestZipPackagePropertiesMarshaller.java
UU
src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/TestContentTypeManager.java
UUsrc/ooxml/java/org/apache/poi/xslf/usermodel/XSLFSimpleShape.java
UU
src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/PackagePropertiesMarshaller.java
U 
src/ooxml/java/org/apache/poi/openxml4j/opc/internal/PackagePropertiesPart.java
U src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java
UUsrc/ooxml/java/org/apache/poi/xssf/streaming/SXSSFEvaluationSheet.java
U 
src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFEvaluationWorkbook.java
UUsrc/ooxml/java/org/apache/poi/xssf/streaming/SXSSFEvaluationCell.java
U src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFEvaluationSheet.java
UUsrc/ooxml/java/org/apache/poi/xssf/usermodel/XSSFEvaluationCell.java
U src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCell.java
U 
src/ooxml/java/org/apache/poi/xssf/usermodel/BaseXSSFEvaluationWorkbook.java
UU
src/ooxml/java/org/apache/poi/xssf/usermodel/BaseXSSFFormulaEvaluator.java
U 
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFEvaluationWorkbook.java
U src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFComment.java
U src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFTable.java
AUsrc/testcases/org/apache/poi/util/TestLocaleUtil.java
U src/testcases/org/apache/poi/ss/formula/TestWorkbookEvaluator.java
U src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java
A src/java/org/apache/poi/util/Removal.java
U src/java/org/apache/poi/util/HexRead.java
U src/java/org/apache/poi/util/BitField.java
U src/java/org/apache/poi/util/LocaleUtil.java
U src/java/org/apache/poi/util/Internal.java
UUsrc/java/org/apache/poi/ss/formula/EvaluationSheet.java
UU
src/java/org/apache/poi/ss/formula/eval/forked/ForkedEvaluationSheet.java
UU
src/java/org/apache/poi/ss/formula/eval/forked/ForkedEvaluationWorkbook.java
UU
src/java/org/apache/poi/ss/formula/eval/forked/ForkedEvaluationCell.java
U src/java/org/apache/poi/ss/formula/WorkbookEvaluator.java
U src/java/org/apache/poi/ss/formula/EvaluationWorkbook.java
UUsrc/java/org/apache/poi/ss/format/CellFormat.java
U src/java/org/apache/poi/ss/util/SheetUtil.java
U src/java/org/apache/poi/ss/usermodel/Cell.java
U src/java/org/apache/poi/ss/usermodel/CellType.java
U src/java/org/apache/poi/ss/usermodel/FormulaEvaluator.java
U src/java/org/apache/poi/hssf/record/Margin.java
U src/java/org/apache/poi/hssf/record/CFRule12Record.java
U src/java/org/apache/poi/hssf/record/CommonObjectDataSubRecord.java
U src/java/org/apache/poi/hssf/record/StandardRecord.java
U src/java/org/apache/poi/hssf/record/ColumnInfoRecord.java
U src/java/org/apache/poi/hssf/record/CellRecord.java
U src/java/org/apache/poi/hssf/record/EndSubRecord.java
U src/java/org/apache/poi/hssf/record/DBCellRecord.java
U src/java/org/apache/poi/hssf/record/SharedValueRecordBase.java
U src/java/org/apache/poi/hssf/record/ExternSheetRecord.java
U src/java/org/apache/poi/hssf/record/BiffHeaderInput.java
U src/java/org/apache/poi/hssf/record/ExtendedFormatRecord.java
U src/java/org/apache/poi/hssf/record/AbstractEscherHolderRecord.java
U src/java/org/apache/poi/hssf/record/BoundSheetRecord.java
U 

Build failed in Jenkins: POI-JDK8 #1124

[Apache Jenkins Server]

See 

Changes:

[onealj] add crude unit test for ZipPackagePropertiesMarshaller

[onealj] convert to junit4

[onealj] whitespace and +svn:eol-style=native

--
Started by an SCM change
[EnvInject] - Loading node environment variables.
Building remotely on ubuntu-us1 (Ubuntu ubuntu ubuntu-us golang-ppa) in 
workspace 
Updating http://svn.apache.org/repos/asf/poi/trunk at revision 
'2016-07-08T10:12:41.862 +'
U test-data/spreadsheet/StructuredReferences.xlsx
U src/java/org/apache/poi/hssf/usermodel/HSSFEvaluationWorkbook.java
UUsrc/java/org/apache/poi/hssf/usermodel/HSSFEvaluationSheet.java
U src/java/org/apache/poi/util/LocaleUtil.java
U src/java/org/apache/poi/util/HexRead.java
UUsrc/java/org/apache/poi/ss/formula/EvaluationSheet.java
UU
src/java/org/apache/poi/ss/formula/eval/forked/ForkedEvaluationWorkbook.java
UU
src/java/org/apache/poi/ss/formula/eval/forked/ForkedEvaluationSheet.java
U src/java/org/apache/poi/ss/formula/WorkbookEvaluator.java
U src/java/org/apache/poi/ss/formula/EvaluationWorkbook.java
U 
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFreeformShape.java
D 
src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/AllOpenXML4JInternalTests.java
AU
src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/TestZipPackagePropertiesMarshaller.java
UU
src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/TestContentTypeManager.java
U 
src/ooxml/testcases/org/apache/poi/ss/formula/TestStructuredReferences.java
UUsrc/ooxml/java/org/apache/poi/xslf/usermodel/XSLFSimpleShape.java
UU
src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/PackagePropertiesMarshaller.java
U 
src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFEvaluationWorkbook.java
UUsrc/ooxml/java/org/apache/poi/xssf/streaming/SXSSFEvaluationSheet.java
U src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFComment.java
U src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFEvaluationSheet.java
U 
src/ooxml/java/org/apache/poi/xssf/usermodel/BaseXSSFEvaluationWorkbook.java
U 
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFEvaluationWorkbook.java
U src/testcases/org/apache/poi/util/TestLocaleUtil.java
U src/testcases/org/apache/poi/ss/formula/TestWorkbookEvaluator.java
Fetching 'https://svn.apache.org/repos/asf/poi/site/src/documentation' at -1 
into '
U src/documentation/content/xdocs/status.xml
At revision 1751877

Fetching 'https://svn.apache.org/repos/asf/poi/site/forrest.properties' at -1 
into '
At revision 1751877

At revision 1751877

No changes for https://svn.apache.org/repos/asf/poi/site/src/documentation 
since the previous build
No changes for http://svn.apache.org/repos/asf/poi/site/forrest.properties 
since the previous build
[EnvInject] - Executing scripts and injecting environment variables after the 
SCM step.
[EnvInject] - Injecting as environment variables the properties content 
LANG=en_US.UTF-8

[EnvInject] - Variables injected successfully.
[POI-JDK8] $ /bin/bash -xe /tmp/hudson2272503001491291465.sh
+ svn status
X   forrest.properties
X   src/documentation
M   
test-data/openxml4j/at.pzp.www_uploads_media_PP_Scheinecker-jdk6error.pptx

Performing status on external item at 'src/documentation':
+ exit 0
[POI-JDK8] $ /home/jenkins/tools/ant/latest/bin/ant -Dcoverage.enabled=true 
clean jenkins
Buildfile: 

clean:
   [delete] Deleting directory 

check-jars:

fetch-jars:

check-ooxml-jars:

fetch-ooxml-jars:

init:
[mkdir] Created dir: 
[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[mkdir] Created dir: 

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

Javen O'Neal  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #4 from Javen O'Neal  ---
I left it open because there's no unit test to verify this was fixed nor
verification from Andrei that this fixed the problem [1].

There isn't currently a unit test class for ZipPackagePropertiesMarshaller and
we don't have a file that can trigger this error.

If Andrei is waiting for the next release to test the change, then we might as
well close it.

[1]
http://apache-poi.1045710.n5.nabble.com/Fail-to-save-stream-tp5723711p5723731.html

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

--- Comment #3 from David North  ---
Javen, can we close this now?

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org