Issue 105653 - Setting a property on a paragraph using UNO will always break
Summary: Setting a property on a paragraph using UNO will always break
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: programming (show other issues)
Version: DEV300m60
Hardware: All All
: P2 Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@sw
URL:
Keywords: regression
Depends on:
Blocks: 99999
  Show dependency tree
 
Reported: 2009-10-06 15:42 UTC by cedric.bosdonnat.ooo
Modified: 2013-08-07 14:44 UTC (History)
1 user (show)

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

Attachments
Patch fixing the issue (981 bytes, patch)
2009-10-06 15:43 UTC, cedric.bosdonnat.ooo 		no flags Details | Diff
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 cedric.bosdonnat.ooo 2009-10-06 15:42:46 UTC 
http://svn.services.openoffice.org/opengrok/xref/Current%20(trunk)/sw/source/core/unocore/unoparagraph.cxx#308

in this code, the sTmp variable is always empty: this causes the
SwXTextCursor::SetPropertyValue to raise an exception in every case.

Here is a sample of UNO Basic code showing the problem:
Sub Main
	oEnum = ThisComponent.Text.createEnumeration
	oPara = oEnum.nextElement( )
	
	oPara.setPropertyValue( "PageDescName", "Landscape" )
End Sub

I'll attach a patch fixing the issue
Comment 1 cedric.bosdonnat.ooo 2009-10-06 15:43:36 UTC 
Created attachment 65178 [details]
Patch fixing the issue
Comment 2 michael.ruess 2009-10-06 16:23:57 UTC 
MRU->OS: please take care of this patch proposal.
This issue is a regression from OOo 3.1.1 to 3.2.
Comment 3 openoffice 2009-10-07 10:44:29 UTC 
applied patch and checked
Comment 4 Oliver Specht 2009-11-17 14:10:00 UTC 
Reassigned for verification
Comment 5 michael.ruess 2009-11-23 10:21:55 UTC 
Verified fix in CWS hb32showstoppers2.
Macro now correctly works and changes page orientation into "Landscape".
Comment 6 michael.ruess 2010-01-28 10:49:58 UTC 
Checked fix in DEV300m70 and OOO320m11.