[Bug 59776] New: ZipPackagePropertiesMarshaller exception handling swallows root cause

classic Classic list List threaded Threaded
10 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 59776] New: ZipPackagePropertiesMarshaller exception handling swallows root cause

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

            Bug ID: 59776
           Summary: ZipPackagePropertiesMarshaller exception handling
                    swallows root cause
           Product: POI
           Version: 3.15-dev
          Hardware: PC
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: OPC
          Assignee: [hidden email]
          Reporter: [hidden email]
                CC: [hidden email]

Reported by Andrei Ivanov on user mailing list [1].
> But I still think it would be nice for the root exception to be
> preserved, for future cases, to avoid guessing.

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

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

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

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

--- Comment #1 from Javen O'Neal <[hidden email]> ---
Include cause in OpenXML4JException in r1750893.

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

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

--- Comment #2 from Andrei Ivanov <[hidden email]> ---
Wow, amazing response.
Thank you :-)

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

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

--- Comment #3 from David North <[hidden email]> ---
Javen, can we close this now?

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

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

Javen O'Neal <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #4 from Javen O'Neal <[hidden email]> ---
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: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

Abhishek Kumar <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #5 from Abhishek Kumar <[hidden email]> ---
Hi Team,

I am still facing the issue again,after upgrading my poi version to 3.15.

Fail to save: an error occurs while saving the package : The part
/docProps/app.xml fail to be saved in the stream with marshaller
org.apache.poi.openxml4j.opc.internal.marshallers.DefaultMarshaller@3224e66a

Caused by: org.apache.poi.openxml4j.exceptions.OpenXML4JException: The part
/docProps/app.xml fail to be saved in the stream with marshaller
org.apache.poi.openxml4j.opc.internal.marshallers.DefaultMarshaller@3224e66a
        at
org.apache.poi.openxml4j.opc.ZipPackage.saveImpl(ZipPackage.java:440)
        at org.apache.poi.openxml4j.opc.OPCPackage.save(OPCPackage.java:1375)
        at org.apache.poi.POIXMLDocument.write(POIXMLDocument.java:179)

Request your help.

Thanks-
Abhishek

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

--- Comment #6 from kumai <[hidden email]> ---
Hi,

I`m facing the same issue. My poi version is v3.15. I do not know how can I
reproduce this problem.
Does anyone know how to reproduce this problem?
Should I downgrade to v3.14?

Best regards,
Ryosuke Kumai.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

--- Comment #7 from PJ Fanning <[hidden email]> ---
Hi Ryosuke,
Can you provide a test case to help us debug and hopefully fix the problem?

Regards,
PJ

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

Javen O'Neal <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Javen O'Neal <[hidden email]> ---
This scope of this bug is to not swallow the exception cause. This was fixed in
r1750893, which was included in 3.15 beta 3 and future releases.

The issues from comment 5 and comment 6 appear to be for reasons other than a
swallows exception cause. Please open up new bugs for different issues.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 59776] ZipPackagePropertiesMarshaller exception handling swallows root cause

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=59776

--- Comment #9 from kumai <[hidden email]> ---
Hi PJ,

I made a mistake.
Sorry, I am in trouble because I do not know how to reproduce it. I created the
following tickets.

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

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]