Issue 92568 - Range chooser does not work anymore
Summary: Range chooser does not work anymore
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P2 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
Keywords: oooqa, regression
: 92825 93199 (view as issue list)
Depends on:
Blocks: 88888
  Show dependency tree
Reported: 2008-08-07 18:33 UTC by andreschnabel
Modified: 2013-02-24 21:22 UTC (History)
10 users (show)

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


Note You need to log in before you can comment on or make changes to this issue.
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. ***