Issue 21558 - Incorrect exception thrown from XText::insertTextContent()
Summary: Incorrect exception thrown from XText::insertTextContent()
Status: ACCEPTED
Alias: None
Product: App Dev
Classification: Unclassified
Component: api (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: Other Linux, all
: P5 (lowest) Trivial
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-10-22 15:05 UTC by con.hennessy
Modified: 2013-02-24 21:07 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description con.hennessy 2003-10-22 15:05:10 UTC 
XText::insertTextContent() is defined as being able to throw a 
com::sun::star::lang::IllegalArgumentException  but if I use the code 
    model.Text.insertTextContent( controller.getViewCursor(), table, False); 
where the controller.getViewCursor() is for for model.Text ( e.g. it is for an element 
inside a table in the model.Text then the following expection happens : 
uno.com.sun.star.uno.RuntimeException: text interface and cursor not related 
 
Although the exception is useful/readbale, it should really be an 
IllegalArgumentException
Comment 1 ooo 2003-10-23 09:26:53 UTC 
implementation defect => forwarding to reponsible developer
Comment 2 thomas.lange 2003-10-28 12:32:47 UTC 
Priority lowered to 5.
Comment 3 chne 2003-10-28 12:39:36 UTC 
cn: target to OOo later
Comment 4 con.hennessy 2003-10-28 19:13:09 UTC 
Please explain why the priority has been set to 5 
And why this *simple* bug ( yes I've looked at the code ), 
cannot be fixed in 1.1 series ?
Comment 5 thomas.lange 2003-10-29 09:22:06 UTC 
Because there is no functionality missing or broken.
It is only a matter of *wrong* type of exception, an exception is
thrown anyway...

Thus it has rather low priority compared to other bugs where
functionality or documentation is really missing or broken.
You surely more like to have one of those (for example your
compareRegiongStart issue) fixed instead of 10 issues like this one.

And if you really think this one is that important. You should
describe why you need to have it fixed.
Also if you find it easy enough to fix you're welcome to provide a
patch by yourself.
Comment 6 versurppersio 2010-11-11 03:17:12 UTC 
Created attachment 74110