You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2016/07/01 09:01:08 UTC

[Bug 59775] New: XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

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

            Bug ID: 59775
           Summary: XSSFHyperlink has wrong type if it contains a location
                    (CTHyperlink#getLocation)
           Product: POI
           Version: 3.12-FINAL
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSSF
          Assignee: dev@poi.apache.org
          Reporter: alvarezg@gmail.com

Created attachment 33998
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=33998&action=edit
Sample test file

Tested with attached file created in MS Excel, hyperlinks for URLs containing
'#' char are stored using both target and location properties. But
XSSFHyperlink sets type to Hyperlink.LINK_DOCUMENT if location is not null.

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

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

[Bug 59775] XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59775

--- Comment #1 from Guillermo Alvarez <al...@gmail.com> ---
Created attachment 33999
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=33999&action=edit
Proposed solution for XSSFHyperlink

patch created with git format-patch, hope the format is valid

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

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

[Bug 59775] XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59775

Guillermo Alvarez <al...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

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

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

[Bug 59775] [PATCH] XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59775

--- Comment #3 from Guillermo Alvarez <al...@gmail.com> ---
Great news, thanks!

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

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

[Bug 59775] [PATCH] XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59775

Dominik Stadler <do...@gmx.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable

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

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

[Bug 59775] [PATCH] XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59775

Guillermo Alvarez <al...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|XSSFHyperlink has wrong     |[PATCH] XSSFHyperlink has
                   |type if it contains a       |wrong type if it contains a
                   |location                    |location
                   |(CTHyperlink#getLocation)   |(CTHyperlink#getLocation)

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

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

[Bug 59775] [PATCH] XSSFHyperlink has wrong type if it contains a location (CTHyperlink#getLocation)

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59775

Javen O'Neal <on...@apache.org> changed:

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

--- Comment #2 from Javen O'Neal <on...@apache.org> ---
Thanks for the patch! Applied in r1753013 with a unit test.

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

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