Bug 334093

Hi Robert, could you confirm this is the following scenario?

1. Start creating a new form, do not close the design
2. Create a new query Q
3. Go back to the form design

Result: form Q is listed but not properly ordered within the list.

Expected result: form Q is listed and ordered within the list.

Created attachment 86555 [details]
A name of a newly added query does not appear on proper position on the list.

Just checked, the same applies a newly added table.

No feedback from the reporter so for now I am changing the summary so first visible, directly related and obvious defects can get fixed. 

@Robert If you meant any unrelated bug we can create another report.

Git commit f3949bc88b3a3266980c7e405617dac6a3360c06 by Wojciech Kosowicz.
Committed on 03/06/2014 at 21:24.
Pushed by wkosowicz into branch 'calligra/2.8'.

Change of addItem to insertItem in KexiDataSourceComboBox slot

In slotNewItemStored of KexiDataSourceComboBox there was called addItem
method that adds to the end of combobox. Method insertItem should be
called instead so that items are in correct order
FIXED-IN: 2.8.4
REVIEW: 118485

M  +2    -2    kexi/widget/KexiDataSourceComboBox.cpp

http://commits.kde.org/calligra/f3949bc88b3a3266980c7e405617dac6a3360c06

Git commit 5a4e7a9e04273635cd50899f4104c51e37b0a045 by Wojciech Kosowicz.
Committed on 03/06/2014 at 21:24.
Pushed by wkosowicz into branch 'master'.

Change of addItem to insertItem in KexiDataSourceComboBox slot

In slotNewItemStored of KexiDataSourceComboBox there was called addItem
method that adds to the end of combobox. Method insertItem should be
called instead so that items are in correct order
FIXED-IN: 2.8.4
REVIEW: 118485

M  +2    -2    kexi/widget/KexiDataSourceComboBox.cpp

http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045

Wojciech,
Nice, just one minor remark. The commit message should better mention the
summary of the bug in the 1st line, i.e.
 "Properly sort newly added tables and queries in forms data source combo"
And primarily, it should explain *why* it exists, *not* what is being done.

Always think about commit message as about input to the official changelog.


Thiago has explained it nicely:
https://plus.google.com/u/0/+ThiagoMacieira/posts/CF5VKSnXXTT



On 3 June 2014 23:33, Wojciech Kosowicz <pcellix@gmail.com> wrote:

>  Wojciech Kosowicz <pcellix@gmail.com> changed bug 334093
> <https://bugs.kde.org/show_bug.cgi?id=334093>
>  What Removed Added  Latest Commit
> http://commits.kde.org/calligra/f3949bc88b3a3266980c7e405617dac6a3360c06
> http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045
>
>  *Comment # 6 <https://bugs.kde.org/show_bug.cgi?id=334093#c6> on bug
> 334093 <https://bugs.kde.org/show_bug.cgi?id=334093> from Wojciech Kosowicz
> <pcellix@gmail.com> *
>
> Git commit 5a4e7a9e04273635cd50899f4104c51e37b0a045 by Wojciech Kosowicz.
> Committed on 03/06/2014 at 21:24.
> Pushed by wkosowicz into branch 'master'.
>
> Change of addItem to insertItem in KexiDataSourceComboBox slot
>
> In slotNewItemStored of KexiDataSourceComboBox there was called addItem
> method that adds to the end of combobox. Method insertItem should be
> called instead so that items are in correct order
> FIXED-IN: 2.8.4
> REVIEW: 118485
>
> M  +2    -2    kexi/widget/KexiDataSourceComboBox.cpp
> http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045
>
>  ------------------------------
> You are receiving this mail because:
>
>    - You are on the CC list for the bug.
>    - You are watching someone on the CC list of the bug.
>    - You are watching the assignee of the bug.
>
>