Issue 92568

Summary:	Range chooser does not work anymore
Product:	General	Reporter:	andreschnabel <andre.schnabel>
Component:	chart	Assignee:	kla <thomas.klarhoefer>
Status:	CLOSED FIXED	QA Contact:	issues@graphics <issues>
Severity:	Trivial
Priority:	P2	CC:	IngridvdM, issues, jbf.faure, josef.latt, nesshof, niklas.nebel, oc, thomas.benisch, thomas.klarhoefer, uwe.luebbers
Version:	3.3.0 or older (OOo)	Keywords:	oooqa, regression
Target Milestone:	---
Hardware:	All
OS:	All
Issue Type:	DEFECT	Latest Confirmation in:	---
Developer Difficulty:	---
Issue Depends on:
Issue Blocks:	88888

Description andreschnabel 2008-08-07 18:33:18 UTC

If a chart is based on a spraedsheet, it is not possible to select a data range
via mouse anymore. Only single cells can be selected.

To reproduce:
- create a new spreadsheet document
- create a new chart, using the chart wizard
- in step 2 of the wizard, klick on the shrink button
- select a datarange (e.g. A1:C7)

expected result: data range (A1:C7) is written to the edit box
current behavior: only the first cell of the selected range (A1) is written to
the edit box 

happens also with OOO300_m1

Comment 1 andreschnabel 2008-08-07 18:46:02 UTC

regression from OOo 3.0beta1 / OOo 2.4

the probel is at any place in the chart module, where you can select data ranges
(e.g. for data series)

Comment 2 saswtr 2008-08-18 01:50:21 UTC

Reproducible in OOO300m2 (WinXP) !!!

Comment 3 kla 2008-08-18 10:20:21 UTC

Confirmed. IHA pls have a look. Also reproducible in OOO300m1 and DEV300m28.
Last version its work fine maybe m25.

Comment 4 kla 2008-08-18 11:43:01 UTC

*** Issue 92825 has been marked as a duplicate of this issue. ***

Comment 5 IngridvdM 2008-08-19 09:22:50 UTC

In method ScViewFunctionSet::SetCursorAtCell the boolean bRefMode is determined
different than before. When the range chooser is used from within the chart the
boolean is wrongly set to false. This causes a missing update of the range in
the edit field of the range chooser, thus only the first click with the first
cell is taken. The changes in calculation of bRefMode were made for issue 91550
and came in with xmlfilter06.

Comment 6 IngridvdM 2008-08-19 10:14:42 UTC

Fixed in CWS chart31.
@Niklas, please review the changes in sc/source/ui/view/tabvwsha.cxx. Thanks!

Comment 7 IngridvdM 2008-08-19 10:54:47 UTC

I see this a 3.0 show stopper, as the range chooser is an important part of the
chart and this is an regression compared to previous releases -> raising target
to 3.0.

Comment 8 niklas.nebel 2008-08-19 13:39:08 UTC

The change in tabvwsha.cxx looks good.

Comment 9 IngridvdM 2008-08-19 15:05:24 UTC

@Thomas, please verify in CWS chart31. Please make also sure that issue 91550
still does work.

Comment 10 thomas.benisch 2008-08-21 10:26:57 UTC

IHA: Uups, sorry for this regression. I verified your fix in CWS chart31.
The fix for issue #91550 still works.

Comment 11 kla 2008-08-21 10:37:36 UTC

@tbe: No Problem, you are welcome. I have had nearly verified the issue.
Important is that the fix for issue 91550 still works.

Comment 12 kla 2008-08-21 14:01:14 UTC

Seen ok in cws chart31 -> verified

Comment 13 kla 2008-09-02 10:56:10 UTC

Seen ok in current master -> closed

Comment 14 IngridvdM 2008-09-10 09:05:54 UTC

*** Issue 93199 has been marked as a duplicate of this issue. ***