Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


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


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.