Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/2444
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/44/2444/1
Translated German comments and messages to English
Change-Id: I80444aac8fdf90ea0f595151788afb14d4415cd3
---
M dbaccess/source/ui/querydesign/QueryTableView.cxx
1 file changed, 66 insertions(+), 64 deletions(-)
diff --git a/dbaccess/source/ui/querydesign/QueryTableView.cxx
b/dbaccess/source/ui/querydesign/QueryTableView.cxx
index db886aa..dfc44d8 100644
--- a/dbaccess/source/ui/querydesign/QueryTableView.cxx
+++ b/dbaccess/source/ui/querydesign/QueryTableView.cxx
@@ -273,27 +273,29 @@
DBG_CHKTHIS(OQueryTableView,NULL);
TTableWindowData* pTabWinDataList = m_pView->getController().getTableWindowData();
OSL_ENSURE((getTableConnections()->size()==0) && (GetTabWinMap()->size()==0),
- "vor OQueryTableView::ReSync() bitte ClearAll aufrufen !");
+ "before calling OQueryTableView::ReSync() please call ClearAll !");
// ich brauche eine Sammlung aller Fensternamen, deren Anlegen schief geht, damit ich die
entsprechenden Connections
+ // gar nicht erst anlege
+ // I need a collection of all window names, deren Anlegen schief geht, damit ich die
entsprechenden Connections
// gar nicht erst anlege
::std::vector<String> arrInvalidTables;
TTableWindowData::reverse_iterator aIter = pTabWinDataList->rbegin();
- // Fenster kreieren und einfuegen
+ // Create the window and add it
for(;aIter != pTabWinDataList->rend();++aIter)
{
OQueryTableWindowData* pData = static_cast<OQueryTableWindowData*>(aIter->get());
OTableWindow* pTabWin = createWindow(*aIter);
- // ich gehe jetzt NICHT ueber ShowTabWin, da dieses die Daten des Fensters in die Liste
des Docs einfuegt, was
- // schlecht waere, denn genau von dort hole ich sie ja gerade
- // also Schritt fuer Schritt
+ // I dont't use ShowTabWin as this adds the window data to the list of documents.
+ // This would be bad as I am getting them from there.
+ // Instead, I do it step by step
if (!pTabWin->Init())
{
- // das Initialisieren ging schief, dass heisst, dieses TabWin steht nicht zur
Verfuegung, also muss ich es inklusive
- // seiner Daten am Dokument aufraeumen
+ // The initialisation has gone wrong, this TabWin is not available, so
+ // I must clean up the data and the document
pTabWin->clearListBox();
delete pTabWin;
arrInvalidTables.push_back(pData->GetAliasName());
@@ -302,15 +304,15 @@
continue;
}
- (*GetTabWinMap())[pData->GetAliasName()] = pTabWin; // am Anfang einfuegen, da ich die
DataList ja rueckwaerts durchlaufe
- // wenn in den Daten keine Position oder Groesse steht -> Default
+ (*GetTabWinMap())[pData->GetAliasName()] = pTabWin; // add at the beginning as I am going
backwards through the DataList
+ // Use the default if there is no position or size
if (!pData->HasPosition() && !pData->HasSize())
SetDefaultTabWinPosSize(pTabWin);
pTabWin->Show();
}
- // Verbindungen einfuegen
+ // Add the connections
TTableConnectionData* pTabConnDataList = m_pView->getController().getTableConnectionData();
TTableConnectionData::reverse_iterator aConIter = pTabConnDataList->rbegin();
@@ -401,11 +403,11 @@
void OQueryTableView::AddTabWin(const ::rtl::OUString& _rTableName, const ::rtl::OUString&
_rAliasName, sal_Bool bNewTable)
{
DBG_CHKTHIS(OQueryTableView,NULL);
- // das ist die aus der Basisklasse geerbte Methode, die fuehre ich auf die an meinem Parent
zurueck, die mir eventuell einen
- // Alias dazu bastelt und das an mein anderes AddTabWin weiterreicht
+ // This is the method inherited from the base class. I send it back to my parent class and my
parent
+ // may build an Alias and send it on to AddTabWin.
- // leider ist _rTableName voll qualifiziert, das OQueryDesignView erwartet aber einen String,
der
- // nur aus Schema und Tabelle besteht und keinen Katalog enthaelt.
+ // Unfortunately _rTableName is fully qualified, but OQueryDesignView expects a String
consisting
+ // of the Schema and Table only and not containing a Catalog.
Reference< XConnection> xConnection = m_pView->getController().getConnection();
if(!xConnection.is())
return;
@@ -465,10 +467,10 @@
void OQueryTableView::AddTabWin(const ::rtl::OUString& _rComposedName, const ::rtl::OUString&
_rTableName, const ::rtl::OUString& strAlias, sal_Bool bNewTable)
{
DBG_CHKTHIS(OQueryTableView,NULL);
- OSL_ENSURE(!_rTableName.isEmpty() || !strAlias.isEmpty(), "OQueryTableView::AddTabWin : kein
Tabellen- und kein Aliasname !");
- // wenn der Tabellenname nicht gesetzt ist, steht das fuer ein Dummy-Fenster, das braucht
aber wenigstens einen Alias-Namen
+ OSL_ENSURE(!_rTableName.isEmpty() || !strAlias.isEmpty(), "OQueryTableView::AddTabWin : no
tables or aliases !");
+ // If the table is not set, then it is a dummy window, but at least the alias must be set
- // neue Datenstruktur erzeugen
+ // build a new data structure
// first check if this already hav it's data
sal_Bool bAppend = bNewTable;
TTableWindowData::value_type pNewTabWinData;
@@ -485,13 +487,13 @@
bAppend = ( aWinIter == aWinEnd );
if ( bAppend )
pNewTabWinData = createTableWindowData(_rComposedName, _rTableName, strAlias);
- // die TabWinData brauche ich nicht in die entsprechende Liste der DocShell eintragen, das
macht ShowTabWin
+ // I do not need to add TabWinData to the DocShell list, ShowTabWin does that.
- // neues Fenster erzeugen
+ // Create a new window
OQueryTableWindow* pNewTabWin = static_cast<OQueryTableWindow*>(createWindow(pNewTabWinData));
- // das Init kann ich hier weglassen, da das in ShowTabWin passiert
+ // No need to initialize, as that happens in ShowTabWin
- // Neue UndoAction
+ // New UndoAction
OQueryTabWinShowUndoAct* pUndoAction = new OQueryTabWinShowUndoAct(this);
pUndoAction->SetTabWin(pNewTabWin); // Fenster
sal_Bool bSuccess = ShowTabWin(pNewTabWin, pUndoAction,bAppend);
@@ -505,7 +507,7 @@
return;
}
- // Relationen zwischen den einzelnen Tabellen anzeigen
+ // Show the relations between the individual tables
OTableWindowMap* pTabWins = GetTabWinMap();
if(bNewTable && !pTabWins->empty() && !_rTableName.isEmpty())
{
@@ -604,7 +606,7 @@
} while ( false );
}
- // mein Parent brauche ich, da es vom Loeschen erfahren soll
+ // My parent needs to be informed about the delete
m_pView->getController().addUndoActionAndInvalidate( pUndoAction );
if (bSuccess && m_lnkTabWinsChangeHandler.IsSet())
@@ -628,14 +630,14 @@
OTableConnection* pConn = GetTabConn(pSourceWin,pDestWin,true);
if ( !pConn )
{
- // neues Daten-Objekt
+ // new Data object
OQueryTableConnectionData* pNewConnectionData = new
OQueryTableConnectionData(pSourceWin->GetData(), pDestWin->GetData());
TTableConnectionData::value_type aNewConnectionData(pNewConnectionData);
sal_uInt32 nSourceFieldIndex, nDestFieldIndex;
ETableFieldType eSourceFieldType, eDestFieldType;
- // Namen/Position/Typ der beiden betroffenen Felder besorgen ...
+ // Get the name/position/type of both the effected fields...
// Source
nSourceFieldIndex = jxdSource.pListBox->GetModel()->GetAbsPos(jxdSource.pEntry);
@@ -646,7 +648,7 @@
nDestFieldIndex = jxdDest.pListBox->GetModel()->GetAbsPos(jxdDest.pEntry);
eDestFieldType = static_cast<
OTableFieldInfo*>(jxdDest.pEntry->GetUserData())->GetKeyType();
- // ... und setzen
+ // ... and set them
pNewConnectionData->SetFieldIndex(JTCS_FROM, nSourceFieldIndex);
pNewConnectionData->SetFieldIndex(JTCS_TO, nDestFieldIndex);
@@ -658,7 +660,7 @@
OQueryTableConnection aNewConnection(this, aNewConnectionData);
NotifyTabConnection(aNewConnection);
- // wie immer bei NotifyTabConnection ist das Verwenden lokaler Variablen unkritisch,
da sowieso eine Kopie erzeugt wird
+ // As usual with NotifyTabConnection, using a local variable is fine because a copy is
made
}
else
{
@@ -741,8 +743,8 @@
OQueryTableWindow* OQueryTableView::FindTable(const String& rAliasName)
{
DBG_CHKTHIS(OQueryTableView,NULL);
- OSL_ENSURE(rAliasName.Len(), "OQueryTableView::FindTable : der AliasName sollte nicht leer
sein !");
- // (nicht dass es schadet, aber es ist sinnlos und weist vielleicht auf Fehler beim
Aufrufer hin)
+ OSL_ENSURE(rAliasName.Len(), "OQueryTableView::FindTable : the AliasName should not be empty
!");
+ // (it is harmless but does not make sense and indicates that there is probably an error
in the caller)
OTableWindowMap::const_iterator aIter = GetTabWinMap()->find(rAliasName);
if(aIter != GetTabWinMap()->end())
return static_cast<OQueryTableWindow*>(aIter->second);
@@ -771,20 +773,20 @@
DBG_CHKTHIS(OQueryTableView,NULL);
OSL_ENSURE(pTabWin != NULL, "OQueryTableView::RemoveTabWin : Fenster sollte ungleich NULL sein
!");
- // mein Parent brauche ich, da es vom Loeschen erfahren soll
+ // I need my parent so it can be info,ed abou the deletion
OQueryDesignView* pParent = static_cast<OQueryDesignView*>(getDesignView());
SfxUndoManager& rUndoMgr = m_pView->getController().GetUndoManager();
rUndoMgr.EnterListAction( String( ModuleRes(STR_QUERY_UNDO_TABWINDELETE) ), String() );
- // Undo-Action anlegen
+ // add the Undo-Action
OQueryTabWinDelUndoAct* pUndoAction = new OQueryTabWinDelUndoAct(this);
pUndoAction->SetTabWin(static_cast< OQueryTableWindow*>(pTabWin));
- // und Fenster verstecken
+ // and hide the window
HideTabWin(static_cast< OQueryTableWindow*>(pTabWin), pUndoAction);
- // Undo Actions und Loeschen der Felder in SelectionBrowseBox
+ // Undo Actions and delete the fields in SelectionBrowseBox
pParent->TableDeleted( static_cast<
OQueryTableWindowData*>(pTabWin->GetData().get())->GetAliasName() );
m_pView->getController().addUndoActionAndInvalidate( pUndoAction );
@@ -817,7 +819,7 @@
void OQueryTableView::GetConnection(OQueryTableConnection* pConn)
{
DBG_CHKTHIS(OQueryTableView,NULL);
- // bei mir und dem Dokument einfuegen
+ // add to me and the document
addConnection( pConn );
}
@@ -826,8 +828,8 @@
void OQueryTableView::DropConnection(OQueryTableConnection* pConn)
{
DBG_CHKTHIS(OQueryTableView,NULL);
- // Selektion beachten
- // bei mir und dem Dokument rausnehmen
+ // Pay attention to the selction
+ // remove from me and the document
RemoveConnection( pConn ,sal_False);
}
@@ -836,15 +838,15 @@
{
DBG_CHKTHIS(OQueryTableView,NULL);
OTableWindowMap* pTabWins = GetTabWinMap();
- OSL_ENSURE(pTabWins != NULL, "OQueryTableView::HideTabWin : habe keine TabWins !");
+ OSL_ENSURE(pTabWins != NULL, "OQueryTableView::HideTabWin : have no TabWins !");
if (pTabWin)
{
- // Fenster
- // die Position in seinen Daten speichern
+ // Window
+ // save the position in its data
getDesignView()->SaveTabWinUIConfig(pTabWin);
- // (ich muss ueber das Parent gehen, da nur das die Position der Scrollbars kennt)
- // dann aus der Liste der TabWins raus und verstecken
+ // (I need to go via the parent, as only the parent knows the position of the scrollbars)
+ // and then out of the TabWins list and hide
OTableWindowMap::iterator aIter = pTabWins->begin();
OTableWindowMap::iterator aEnd = pTabWins->end();
for ( ;aIter != aEnd ; ++aIter )
@@ -854,20 +856,20 @@
break;
}
- pTabWin->Hide(); // nicht zerstoeren, steht im Undo!!
+ pTabWin->Hide(); // do not destroy it, as it is still in the undo list!!
- // die Daten zum TabWin muessen auch aus meiner Verantwortung entlassen werden
+ // the TabWin data must also be passed out of my responsibility
TTableWindowData* pTabWinDataList = m_pView->getController().getTableWindowData();
pTabWinDataList->erase(
::std::remove(pTabWinDataList->begin(),pTabWinDataList->end(),pTabWin->GetData()),pTabWinDataList->end());
- // NICHT loeschen, da ja das TabWin selber - das noch lebt - sie auch noch braucht
- // Entweder geht es irgendwann wieder in meine Verantwortung ueber, (ueber
ShowTabWin), dann fuege ich
- // auch die Daten wieder ein, oder die Undo-Action, die im Augenblick die alleinige
Verantwortung fuer das Fenster
- // und dessen Daten hat, wird zestoert, dann loescht es beides
+ // The data should not be destroyed as TabWin itself - which is still alive - needs
them
+ // Either it goes back into my responsibility, (via ShowTabWin), then I add the data
back,
+ // or the Undo-Action, which currently has full responsibility for the window
+ // and its data, gets destroyed and destroys both the window and its data
if (m_pLastFocusTabWin == pTabWin)
m_pLastFocusTabWin = NULL;
- // Verbindungen, die zum Fenster gehoeren, einsammeln und der UndoAction uebergeben
+ // Collect the connections which belong to the window and pass them to the UndoAction
sal_Int16 nCnt = 0;
const ::std::vector<OTableConnection*>* pTabConList = getTableConnections();
::std::vector<OTableConnection*>::const_iterator aIter2 = pTabConList->begin();
@@ -896,10 +898,10 @@
m_pView->getController().InvalidateFeature(ID_BROWSER_ADDTABLE);
- // der UndoAction sagen, dass das Fenster (inklusive der Connections) jetzt in seinem
Besitzt ist
+ // inform the UndoAction that thewindow and connections belong to it
pUndoAction->SetOwnership(sal_True);
- // damit habe ich das Doc natuerlich modifiziert
+ // by doing so, we have modified the document
m_pView->getController().setModified( sal_True );
m_pView->getController().InvalidateFeature(SID_BROWSER_CLEAR_QUERY);
}
@@ -917,18 +919,18 @@
if (pTabWin->Init())
{
TTableWindowData::value_type pData = pTabWin->GetData();
- OSL_ENSURE(pData != NULL, "OQueryTableView::ShowTabWin : TabWin hat keine Daten !");
- // Wenn die Daten schon PosSize haben, diese benutzen
+ OSL_ENSURE(pData != NULL, "OQueryTableView::ShowTabWin : TabWin has no data !");
+ // If there is a position and size defined, we use them
if (pData->HasPosition() && pData->HasSize())
{
Size aSize(CalcZoom(pData->GetSize().Width()),CalcZoom(pData->GetSize().Height()));
pTabWin->SetPosSizePixel(pData->GetPosition(), aSize);
}
else
- // ansonsten selber eine Default-Position ermitteln
+ // else set a default position
SetDefaultTabWinPosSize(pTabWin);
- // Fenster zeigen und in Liste eintragen
+ // Show the window and add to the list
::rtl::OUString sName = static_cast<
OQueryTableWindowData*>(pData.get())->GetAliasName();
OSL_ENSURE(GetTabWinMap()->find(sName) == GetTabWinMap()->end(),"Alias name already in
list!");
GetTabWinMap()->insert(OTableWindowMap::value_type(sName,pTabWin));
@@ -936,12 +938,12 @@
pTabWin->Show();
pTabWin->Update();
- // Das Update ist notwendig, damit die Connections an dem Fenster richtig
gezeichnet werden. Klingt absurd,
- // ich weiss. Aber die Listbox haelt sich intern ein Member, was bei ersten
Zeichnen (nachdem die Listbox im Init
- // gerade neu gefuellt wurde) initialisiert wird, und genau dieses Member wird
irgendwann benoetigt fuer
- // GetEntryPos, und dieses wiederum von der Connection, wenn sie ihren Ansatzpunkt
am Fenster feststellen will.
+ // We must call Update() in order to show the connections in the window correctly.
This sounds strange,
+ // but the Listbox has an internal Member which is initialized when the Listbox is
first shown (after the Listbox
+ // is filled in Init). This Member will eventually be needed for
+ // GetEntryPos, and then in turn by the Connection, when its starting point to the
window must be determined.
- // die Connections
+ // the Connections
::std::vector<OTableConnection*>* pTableCon = pUndoAction->GetTabConnList();
::std::vector<OTableConnection*>::iterator aIter = pTableCon->begin();
::std::vector<OTableConnection*>::iterator aEnd = pTableCon->end();
@@ -951,13 +953,13 @@
pTableCon->clear();
- // und die Daten des Fensters ebenfalls in Liste (des Docs)
+ // and add the window's data to the list (of the document)
if(_bAppend)
m_pView->getController().getTableWindowData()->push_back(pTabWin->GetData());
m_pView->getController().InvalidateFeature(ID_BROWSER_ADDTABLE);
- // und der UndoAction sagen, dass das Fenster jetzt meine ist ...
+ // and inform the UndoAction that the window belongs to me
pUndoAction->SetOwnership(sal_False);
bSuccess = sal_True;
@@ -965,8 +967,8 @@
else
{
//////////////////////////////////////////////////////////////////
- // Initialisierung fehlgeschlagen
- // (z.B. wenn Verbindung zur Datenbank in diesem Augenblick unterbrochen worden ist)
+ // Initiaisation failed Initialisierung fehlgeschlagen
+ // (for example when the Connection to the database is not available at the moment)
pTabWin->clearListBox();
delete pTabWin;
}
@@ -984,7 +986,7 @@
void OQueryTableView::InsertField(const OTableFieldDescRef& rInfo)
{
DBG_CHKTHIS(OQueryTableView,NULL);
- OSL_ENSURE(getDesignView() != NULL, "OQueryTableView::InsertField : habe kein Parent !");
+ OSL_ENSURE(getDesignView() != NULL, "OQueryTableView::InsertField : has no Parent !");
static_cast<OQueryDesignView*>(getDesignView())->InsertField(rInfo);
}
//------------------------------------------------------------------------------
--
To view, visit https://gerrit.libreoffice.org/2444
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I80444aac8fdf90ea0f595151788afb14d4415cd3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: David Verrier <dverrier@gmail.com>
Context
- [PATCH] Translated German comments and messages to English · David Verrier (via Code Review)
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.