[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2008-11-08 Thread mechtilde
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User mechtilde changed the following:

What|Old value |New value

   Status whiteboard|  |dba24a





-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-09-10 Thread clu
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User clu changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from [EMAIL PROTECTED] Mon Sep 10 15:04:16 + 
2007 ---
verified in cws

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-09-04 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|fs|msc





--- Additional comments from [EMAIL PROTECTED] Tue Sep  4 07:13:01 + 
2007 ---
fs-> msc: please verify in CWS dba24a

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-09-03 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

Target milestone|OOo Later |OOo 2.4





--- Additional comments from [EMAIL PROTECTED] Mon Sep  3 10:37:10 + 
2007 ---
targeting to 2.4, since the fix is part of a CWS aiming for this release

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-23 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|dyf   |fs





--- Additional comments from [EMAIL PROTECTED] Mon Jul 23 12:27:01 + 
2007 ---
patch committed to CWS dba24a. Thanks for submitting it!

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-23 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

  Status|STARTED   |RESOLVED

  Resolution|  |FIXED





--- Additional comments from [EMAIL PROTECTED] Mon Jul 23 12:04:33 + 
2007 ---
I committed the patch to CWS dba24a. Attached is the complete patch I committed,
you might want to compare it to your last patch - I did three changes:
- I removed some unnecessary whitespace changes
- in OCopyTable::SetAppendDataRadio, I re-inserted the call to
  "m_pParent->setCreateStyle". This is strictly necessary, else, pasting a
  table while there is no selection, and *then* selecting "Append data"
  in the wizard will lead to wrong results.
- I did changes to OCopyTable::setCreateStyleAction, so that it now also
  calls RadioChangeHdl where necessary. This ensures that if later on,
  RadioChangeHdl is changed in some way (for instance with en/disabling
  more controls), then this is automatically inherited by
  setCreateStyleAction.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-23 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |46998)
patch as committed 
|  |to CWS dba24a






--- Additional comments from [EMAIL PROTECTED] Mon Jul 23 12:00:06 + 
2007 ---
Created an attachment (id=46998)
patch as committed to CWS dba24a


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-23 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |46989)
new patch






--- Additional comments from [EMAIL PROTECTED] Mon Jul 23 09:16:33 + 
2007 ---
Created an attachment (id=46989)
new patch


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-22 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jul 23 06:36:23 + 
2007 ---
well, my idea was the current code works fine, but it is easy to break it in
future. That is, we're talking about robustness of code here.

Consider the example I mentioned: if tomorrow, somebody does a
|setCreateStyleAction( WIZARD_DEF )| at the page, then he needs to also do a
|setCreateStyle| at the dialog. (Sorry, I mistyped this in my previous comment,
I wrongly wrote "m_pParent->setCreateStyleAction" instead of
"m_pParent->setCreateStyle".).

This coupling is prone to errors. Your new code would be more robust if the
implementation of |setCreateStyleAction| *atuomatically* would call
|setCreateStyle| at the dialog.

Okay, be it. If you would please provide me the complete actual patch, I'm
willing to check it in.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-19 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

  Status|NEW   |STARTED





--- Additional comments from [EMAIL PROTECTED] Fri Jul 20 04:57:54 + 
2007 ---
Hi,Mr Frank:
:)Maybe I havn't understood your ideas.I think OCopyTable is only a page of 
OCopyTableWizard, so setCreateStyleAction is only used to set the check 
boxes's state in  OCopyTable,and SetAppendDataRadio is also only used to set 
the check boxs's state interrelated with append style.

so I had altered some of the following in OCopyTable:

void OCopyTable::setCreateStyleAction()//const 
OCopyTableWizard::Wizard_Create_Style& _eStyle
{
 // reselect the last action before
 switch(m_pParent->getCreateStyle())
 {
...
  case OCopyTableWizard::WIZARD_APPEND_DATA:
   m_aRB_AppendData.Check(sal_True);
   //m_pParent->EnableButton(OCopyTableWizard::WIZARD_NEXT,sal_False);
   SetAppendDataRadio();
...
},

in tablecopyhelper.cxx Changes are as follows :

aWizard.fillTypeInfo();
aWizard.loadData();
if(GetIsSelectCopytable())
{
 aWizard.ResetsName( GetDefaultTableName());
 aWizard.setCreateStyle(OCopyTableWizard::WIZARD_APPEND_DATA);
}

OCopyTable*   pPage1;
pPage1 = new OCopyTable(&aWizard,COPY, bIsView);
pPage1->setCreateStyleAction();
,

and I changed *SetAppendDataRadio* protected.  

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-17 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Tue Jul 17 09:10:47 + 
2007 ---
ok.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-16 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|fs|dyf





--- Additional comments from [EMAIL PROTECTED] Mon Jul 16 08:38:31 + 
2007 ---
okay, patch looks much better now :)

Two tiny little wishes, though:
- with setCreateStyleAction, you introduce a new method, which works fine, but
  is prone to future errors. That is, if somebody later finds this method
useful, and
  uses it for own purposes, the dialog/page might be left inconsistent: Consider
  somebody calling setCreateStyleAction( WIZARD_DEF ) - this would *not*
  be reflected in the dialog's CreateStyle
  In other words, I think setCreateStyleAction is missing a call to
  m_pParent->setCreateStyleAction.
  Speaking strictly, it is also missing a call to SetAppendDataRadio 
respectively
  RadioChangeHdl - those methods would also ensure the state of the remaining
  controls on the page fits the state of the check boxes.
- Please clean you patch from unnecessary whitespace changes (there are a
  few chunks which contain only changes in whitespaces), and unnecessary
  changes such as the new-but-commented-out line 140 in TableCopyHelper.hxx
Thanks.

Those two things addressed, I am willing to commit the patch.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-12 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Fri Jul 13 01:55:06 + 
2007 ---
ok.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-12 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|dyf   |fs

  Status|STARTED   |NEW





--- Additional comments from [EMAIL PROTECTED] Thu Jul 12 19:36:14 + 
2007 ---
I'll have a look at it, thank yoou very much. However, it might take 'til
beginning of next week until I find time, sorry.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-11 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Thu Jul 12 04:58:04 + 
2007 ---
Hi, Frank

The new patch file was made According the way you told me.Thank you :)!Is it 
that you need?

Best Regards.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-07-10 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |46685)
new patch






--- Additional comments from [EMAIL PROTECTED] Wed Jul 11 05:12:32 + 
2007 ---
Created an attachment (id=46685)
new patch


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-26 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Wed Jun 27 05:39:03 + 
2007 ---
better, not yet there:

You now have a "new OCopyTable( ..., OCopyTableWizard::WIZARD_APPEND_DATA )
in two places, and you make this dependent on whether or not the
DefaultTableName is not empty.

The consequence:
OCopyTableWizard::m_eCreateStyle is not in sync with what you see in the UI.
That is, do the following:
- copy a table to the clipboard
- paste it, while it is selected
=> the wizard opens, and "Append" is pre-selected (which is fine)
- press "Next"
=> you get an error saying that the table name is already in use - which is
wrong, since you do not want to create this table, but append to it.

Please fix this scenario.

(side note: thanks for the all-in-one-file patch. Unfortunately, it still
requires multiple manual steps to apply it to a source tree. What I tried to
suggest above is something like
> cp -R dbaccess dbaccess.old
> (do changes in dbaccess)
> diff -ru dbaccess.bak dbaccess

Alternatively, if you have CVS access, a "cvs diff -u" in the root of dbaccess
would produce similar results.

With this, you would have one large patch file, which can be applied in one
single step.)

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-26 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |46271)
modified






--- Additional comments from [EMAIL PROTECTED] Wed Jun 27 04:12:58 + 
2007 ---
Created an attachment (id=46271)
modified


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-25 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jun 25 09:32:46 + 
2007 ---
when both are not equal, this usually means they really refer to different
database documents.

However, there's one scenario where they're not equal, but refer to the same
document: If one is the programmatic name (such as "Bibliography"), and the
other is the file URL (such as "file:///.../biblio.odb") which is registered for
this programmatic name. This should happen relatively seldom, but is not 
impossible.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-25 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jun 25 09:26:02 + 
2007 ---
en, "Ranging from what time" means "when not equals".:)

Because there is a judgement in OTableCopyHelper::insertTable:

if ( _sSrcDataSourceName == _sDestDataSourceName )
xSrcConnection = _xDestConnection;

.And this affect the default table name.
so I want to make it clear,then decided how to modified.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-25 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jun 25 08:43:29 + 
2007 ---
Both have effectively the same semantics. The different names are for historical
reasons only.

In general, a data source name can either be a "registered named" (see the
com.sun.star.sdb.DatabaseContext service, and Tools/Options/Base/Registered
Databases), that is, a programmatic name which internally refers to some URL of
the .odb file.
Or, a data source name can the the URL directly (file:///some/path/to/file.odb).

Both should be freely interchangeable. A data transfer source is free to insert
any of both into the data access descriptor. Not sure which one is used by the
application controller, in theory, it is also free to use both.

I assume your question's background is you have a conflict with those two names
(one being a programmatic name, one being the URL)?

Also, what do you mean with "ranging from what time"?

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-25 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jun 25 08:34:25 + 
2007 ---
Hi,Frank

I have a problem. ::rtl::OUString getDatabaseName() in 
\dbaccess\source\ui\app\ : OApplicationController, and ::rtl::OUString 
ODataAccessDescriptor::getDataSource(),what is the difference of their return 
value? When equal, ranging from what time?

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-22 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Fri Jun 22 19:30:35 + 
2007 ---
Looks like a strange heuristics to me, which probably can be removed, and
replaced by setting m_sName to m_sSourceName. In this case, it should be ensured
that "Append" is pre-selected then.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-22 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

  Status|NEW   |STARTED





--- Additional comments from [EMAIL PROTECTED] Fri Jun 22 09:17:52 + 
2007 ---
ok.

I don't know why do this judgment In OCopyTableWizard::OCopyTableWizard? 

if ( m_xSourceConnection == m_xConnection )
{
Reference xSup(m_xConnection,UNO_QUERY_THROW);
m_sName = ::dbtools::createUniqueName(xSup->getTables
(),m_sSourceName,sal_False);
}
 else
m_sName = m_sSourceName;



-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-21 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

  CC|''|'fs'

 Assigned to|fs|dyf





--- Additional comments from [EMAIL PROTECTED] Thu Jun 21 12:28:24 + 
2007 ---
fs->dyf: better, but not yet finished:
- With a recent version, this does not compile on some platforms because
  OTableCopyHelper::pasteTable (the version in TableCopyHelper.cxx:507)
  does have a (new) parameter _sTableName, which it does not use. For some
  compilers, this is a warning, and warnings are reported as errors in
  a regular OOo build.

- Now we have the desired behaviour when we paste, for instance, a Calc
  cell range into a Base document. In this case, the table which is selected
  in Base is used as default table name - fine.
  However, now we have an inconsistency: When you copy a Base table itself
  to the clipboard, and then paste it, then the current selection in the
  application is *not* respected.
  We should fix this case, too, for consistency reasons.

- in DExport.cxx:781, you change the default from WIZARD_DEF_DATA to
  WIZARD_APPEND_DATA. That's not completely what I meant with my comment:
  Now the default is *always* to append, even if the suggested table name
  does not yet exist. For instance, select *no* table in Base, and then
  paste => you will get a dialog which has "Append" pre-selected, but
  a non-existent table name. That's not what we want to have here.

  I suggest you completely remove the last parameter of the OCopyTable
  constructor, and make a reasonable guess inside the constructor: If
  the a table with the given name already exists, then pre-select "Append",
  else pre-select "Create".

  (Alternatively, you can spare this piece of the patch, since it's  not
  part of the original bug.)

Thanks.
(side note: If possible, please next time generate one big file containing the
changes for all files, this makes applying and reviewing on my side much easier.
diff -ru should do. Thanks.)

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-21 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|dyf   |fs

  Status|REOPENED  |NEW





--- Additional comments from [EMAIL PROTECTED] Thu Jun 21 11:47:01 + 
2007 ---
grabbing for review

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-21 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

  Status|RESOLVED  |REOPENED

  Resolution|FIXED |





--- Additional comments from [EMAIL PROTECTED] Thu Jun 21 11:46:25 + 
2007 ---
re-opening. Sorry, an issue is to be set to FIXED as soon as the patch is
committed to CVS, not earlier. See "A Bug's life time" at
http://qa.openoffice.org/issue_handling/index.html.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-21 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

  Status|STARTED   |RESOLVED

  Resolution|  |FIXED





--- Additional comments from [EMAIL PROTECTED] Thu Jun 21 11:21:05 + 
2007 ---
Add patch of AppController.cxx.Modified some places according to Mr Frank's 
proposal.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-21 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Thu Jun 21 11:02:11 + 
2007 ---
Created an attachment (id=46134)
patch for it


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-17 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User dyf changed the following:

What|Old value |New value

  Status|NEW   |STARTED





--- Additional comments from [EMAIL PROTECTED] Mon Jun 18 03:32:54 + 
2007 ---
hi,fs
:)I'm sorry i didn't notice your review comments above.I have notice I missed 
the patch of OApplicationController.And Your proposal is reasonable,thanks!I 
will complete it soon.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-15 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|fs|dyf





--- Additional comments from [EMAIL PROTECTED] Fri Jun 15 11:48:59 + 
2007 ---
fs->dyf: sorry, I forgot to re-assign this issue to you, so I think you didn't
notice my review comments above.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-11 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jun 11 12:46:52 + 
2007 ---
Created an attachment (id=45805)
review notes


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-11 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Mon Jun 11 12:46:20 + 
2007 ---
Okay, first: thanks for the patch!
Second: I'll attach my review notes. Items marked with "+" need to be fixed
(nothing serious, relax :), items with "-" are suggestions for changes you could
do to the patch, I'll not insist on them.

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-11 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Assigned to|oj|fs

  Issue type|DEFECT|PATCH





--- Additional comments from [EMAIL PROTECTED] Mon Jun 11 11:06:30 + 
2007 ---
grabbing for patch review, since OJ is on vacation

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-11 Thread fs
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907


User fs changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |45802)
combined patch for 
|  |easier readability






--- Additional comments from [EMAIL PROTECTED] Mon Jun 11 11:00:08 + 
2007 ---
Created an attachment (id=45802)
combined patch for easier readability


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-06 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Thu Jun  7 06:49:48 + 
2007 ---
Created an attachment (id=45706)
issue18907 patch


-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[dba-issues] [Issue 18907] wrong default table name i n "Copy table"

2007-06-05 Thread dyf
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=18907





--- Additional comments from [EMAIL PROTECTED] Wed Jun  6 05:50:32 + 
2007 ---
I modified the issue.I had started from AppController.cxx:Execute ,get the 
selected table's name through getSelectionElementNames(...),then extended some 
place to transfer it to where it was needed.
 

-
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]