You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openoffice.apache.org by bu...@apache.org on 2021/11/04 01:15:37 UTC
[Issue 128494] New: Match function error when sheets are re-ordered
https://bz.apache.org/ooo/show_bug.cgi?id=128494
Issue ID: 128494
Issue Type: DEFECT
Summary: Match function error when sheets are re-ordered
Product: Calc
Version: 4.1.11
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: Normal
Priority: P5 (lowest)
Component: programming
Assignee: issues@openoffice.apache.org
Reporter: RonBlackwell@sympatico.ca
Target Milestone: ---
Created attachment 87070
--> https://bz.apache.org/ooo/attachment.cgi?id=87070&action=edit
Shows the incorrect match function when sheets are re-ordered
When sheets are re-ordered, the match function should search the range
specified in the formula not one previously associated with the tab. See the
attached spreadsheet for a simple example.
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
Keith N. McKenna <kn...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Ever confirmed|0 |1
CC| |knmc@apache.org
Status|UNCONFIRMED |CONFIRMED
--- Comment #7 from Keith N. McKenna <kn...@apache.org> ---
Set as confirmed by comment #4
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
RonBlackwell <Ro...@sympatico.ca> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #87070|0 |1
is obsolete| |
--- Comment #1 from RonBlackwell <Ro...@sympatico.ca> ---
Created attachment 87072
--> https://bz.apache.org/ooo/attachment.cgi?id=87072&action=edit
Spreadsheet showing how to create match function problems
Follow the instructions in the spreadsheet to create the problems. Don't save
the spreadsheet because you may not be able to fix the errors which show up.
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
RonBlackwell <Ro...@sympatico.ca> changed:
What |Removed |Added
----------------------------------------------------------------------------
Version|4.1.11 |4.1.14
--- Comment #3 from RonBlackwell <Ro...@sympatico.ca> ---
This problem still occurs on 4.1.14. It seems Calc does not associate ranges -
e.g. A2:A4 - with the correct sheet when sheets are re-ordered.
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
--- Comment #5 from RonBlackwell <Ro...@sympatico.ca> ---
This problem report is a follow-on to 127479. It shows that the problem occurs
without any use of the INDIRECT function. If after moving sheet 3 between
sheet 1 and sheet 2, you copy sheet2.b2 and paste it as unformatted text it
into both sheet2.C2 and sheet2.C3 the results still do not match. Other
functions - e.g. COUNTIF - do not appear to have this problemso it may just be
a problem with the match function not re-calculating properly.
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
Matthias Seidel <ms...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mseidel@apache.org
Latest|--- |4.1.4
Confirmation in| |
Version|4.1.14 |4.1.11
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
Peter <pe...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |petko@apache.org
--- Comment #8 from Peter <pe...@apache.org> ---
code in question
main/sc/source/core/tool/interpr1.cxx
line:4349
if I check 4377
else if (GetStackType() == svMatrix)
{
pMatSrc = PopMatrix();
if (!pMatSrc)
{
PushIllegalParameter();
return;
}
}
looking at Index (8347) in comparison
case svMatrix:
{
if (nArea != 1)
SetError(errIllegalArgument);
sal_uInt16 nOldSp = sp;
ScMatrixRef pMat = GetMatrix();
if (pMat)
{
SCSIZE nC, nR;
pMat->GetDimensions(nC, nR);
// Access one element of a vector independent of
col/row
// orientation?
bool bVector = ((nCol == 0 || nRow == 0) && (nC == 1 ||
nR == 1));
SCSIZE nElement = ::std::max(
static_cast<SCSIZE>(nCol),
static_cast<SCSIZE>(nRow));
if (nC == 0 || nR == 0 ||
(!bVector && (static_cast<SCSIZE>(nCol) > nC ||
static_cast<SCSIZE>(nRow) > nR))
||
(bVector && nElement > nC * nR))
PushIllegalArgument();
else if (nCol == 0 && nRow == 0)
sp = nOldSp;
else if (bVector)
{
--nElement;
if (pMat->IsString( nElement))
PushString( pMat->GetString( nElement));
else
PushDouble( pMat->GetDouble( nElement));
}
else if (nCol == 0)
{
ScMatrixRef pResMat = GetNewMat(nC, 1);
if (pResMat)
{
SCSIZE nRowMinus1 = static_cast<SCSIZE>(nRow -
1);
for (SCSIZE i = 0; i < nC; i++)
if (!pMat->IsString(i, nRowMinus1))
pResMat->PutDouble(pMat->GetDouble(i,
nRowMinus1), i, 0);
else
pResMat->PutString(pMat->GetString(i,
nRowMinus1), i, 0);
PushMatrix(pResMat);
}
else
PushIllegalArgument();
}
else if (nRow == 0)
{
ScMatrixRef pResMat = GetNewMat(1, nR);
if (pResMat)
{
SCSIZE nColMinus1 = static_cast<SCSIZE>(nCol -
1);
for (SCSIZE i = 0; i < nR; i++)
if (!pMat->IsString(nColMinus1, i))
pResMat->PutDouble(pMat->GetDouble(nColMinus1,
i), i);
else
pResMat->PutString(pMat->GetString(nColMinus1,
i), i);
PushMatrix(pResMat);
}
else
PushIllegalArgument();
}
else
{
if (!pMat->IsString( static_cast<SCSIZE>(nCol-1),
static_cast<SCSIZE>(nRow-1)))
PushDouble( pMat->GetDouble(
static_cast<SCSIZE>(nCol-1),
static_cast<SCSIZE>(nRow-1)));
else
PushString( pMat->GetString(
static_cast<SCSIZE>(nCol-1),
static_cast<SCSIZE>(nRow-1)));
}
}
}
break;
ohh my...
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
Matthias Seidel <ms...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest|4.1.4 |4.1.14
Confirmation in| |
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
--- Comment #2 from RonBlackwell <Ro...@sympatico.ca> ---
Attachment 87072 clearly shows that the match function does not search the
correct range when sheets are moved. The value returned in cells C2 of Sheet1,
Sheet2 and Sheet3 should always be 1, 2 and 3, respectively, no matter how the
sheets are -rearranged.
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
--- Comment #6 from RonBlackwell <Ro...@sympatico.ca> ---
Created attachment 87182
--> https://bz.apache.org/ooo/attachment.cgi?id=87182&action=edit
Functions INDEX, COUNTIF, etc. operate correctly but MATCH does not.
Maybe someone could compare the code for the MATCH function with the code for
the INDEX function and figure out how to fix the MATCH function. This problem
has been around for a long time now.
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
--- Comment #9 from Peter <pe...@apache.org> ---
I check a fix... maybe this is easy to fix
--
You are receiving this mail because:
You are the assignee for the issue.
[Issue 128494] Match function error when sheets are re-ordered
Posted by bu...@apache.org.
https://bz.apache.org/ooo/show_bug.cgi?id=128494
jeffouille <ph...@wanadoo.fr> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |philip.elec@wanadoo.fr
--- Comment #4 from jeffouille <ph...@wanadoo.fr> ---
Hello,
I confirm with AOO 4.1.13, but sound like a dump of
https://bz.apache.org/ooo/show_bug.cgi?id=127479
Ctrl + Shift + F9 give correct value after move sheet.
--
You are receiving this mail because:
You are the assignee for the issue.