To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=113181 Issue #|113181 Summary|sw: && || ambiguity in docfld.cxx Component|Word processor Version|DEV300m84 Platform|All URL| OS/Version|Linux Status|NEW Status whiteboard| Keywords| Resolution| Issue type|PATCH Priority|P3 Subcomponent|code Assigned to|os Reported by|cmc
------- Additional comments from c...@openoffice.org Fri Jul 16 10:03:28 +0000 2010 ------- in docfld.cxx we have... if( bIsDBMgr && rDoc.GetNewDBMgr()->OpenDataSource( aDBData.sDataSource, aDBData.sCommand )&& GETFLD_ALL == eGetMode || ( GETFLD_CALC & eGetMode && ((SwDBNextSetField*)pFld)->IsCondValid() )) pFormel = &pFld->GetPar1(); precedence rules for this results in ... if ( ( bIsDBMgr && rDoc.GetNewDBMgr()->OpenDataSource(aDBData.sDataSource, aDBData.sCommand) && GETFLD_ALL == eGetMode ) || (GETFLD_CALC & eGetMode && ((SwDBNumSetField*)pFld)->IsCondValid()) ) { pFormel = &pFld->GetPar1(); } maybe that's what we intended, but there's a chance we intended... if ( (bIsDBMgr && rDoc.GetNewDBMgr()->OpenDataSource(aDBData.sDataSource, aDBData.sCommand)) && (GETFLD_ALL == eGetMode || (GETFLD_CALC & eGetMode && ((SwDBNumSetField*)pFld)->IsCondValid())) ) { pFormel = &pFld->GetPar1(); } --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@sw.openoffice.org For additional commands, e-mail: issues-h...@sw.openoffice.org --------------------------------------------------------------------- To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org For additional commands, e-mail: allbugs-h...@openoffice.org