Issue 309

Don't crash when a broken add_executable is found

do not offer explicitely deleted methods for code completion



Again basically done by Milian.

Patch for bug 274301

Arrangement of items in "Navigation" menu not well structured (kdevelop)
Separated the options group wise in Navigation menu by modifying the corresponding rc files.

do not insert spaces in function definition after void or type with no argument name



Hint: basically done by Milian to trick me into being able to fix some of my
problems myself later.

Resizing of code generation wizard pages improved.

Fix crash

Delete the correct pointer...

Remove unused Run > profile menu entry

Currently we had the menu entry but nobody using it. Technically it's
correct because whenever we'd get a profiler it would work automatically
but I'd say that also having the option but not being able to use
it is nothing but frustrating.

Commenting out the code until somebody puts some care into profiling
integration.

Checks if an optional field was instantiated before accessing it

The private field m_konsolewidget is initialized only on some specific cases, so it should be accessed with caution. But there are some places in witch it's being used without checking if it was instantiated or not. This patch fixes those accesses, by checking if it is instantiated or not.

This was laying around in reviewboard approved by Giorgos Tsiapaliokas but uncommited for 5 months

REVIEW: 110288

Don't store pointers ProjectBaseItem in classes

They could be deleted before used. Store the model index instead.

Error message before scan for unsupported depth/colour combinations

Fix click on trash plasmoid when on desktop and widgets are unlocked

Don't register the icon as a draggable with the applet.

This causes events to be intercepted which prevents launching the file
manager when the applet is movable.

FIXED-IN: 4.11.3
REVIEW: 113127

Use the correct variable, instead of the entire QStringList, to fill in
lineedit's text in autoDetectSystemProxy().
When saving for "System Proxy" configuration be sure to write the name
of environment variables and not their values in the configuration file.
Added missing signal connection for mUi.systemProxyRadioButton.
Connect slotChanged() to textEdited() signal, instead of textChanged(),
for systemProxy's lineedits: this avoid to emit changed() when
"Show the value of the envirnment variables" checkbox is clicked.


FIXED-IN: 4.11.3
REVIEW: 113129

don't leak fpstext texture


FIXED-IN: 4.11.3
REVIEW: 113136

Fix Bug 321304 - Leave A Note widget: When note is delivered, blank note also appears

FIXED-IN: 4.11.3

Include "Space" in the keyboard search string

Before this commit, we only added pressed keys to the search string if
they have no other meaning. This means that files containing a Space in
their name could not be searched because Ctrl+Space toggles the
selection state of the current item, and Space alone selects the
current item.

After this commit, Space is added to the search string if

(a) the key press did not have any other effect, i.e., if Ctrl was not
pressed, and the current item is selected already, and
(b) a keyboard search has been started already (to prevent unexpected
effects when pressing Space accidentally - I think that it's rather
uncommon to have files whose names start with a Space - and to make
the unit test simpler).

I modified the unit test of KItemListController, which did not test
keyboard search yet. This uncovered a small problem in
KItemListController::slotChangeCurrentItem() when NoSelection mode is
used. It's not really relevant for anything that is executed inside
Dolphin, but I still fixed it to make the unit test happy.


FIXED-IN: 4.11.3
REVIEW: 113071

Make sure clicking reset works under every circumstance.


FIXED-IN: 4.11.3

fix mouseclick animation repaint areas

REVIEW: 113212

FIXED-IN: 4.11.3

Reload the view if a previously unmounted device is mounted again

The problem was that DolphinViewContainer::setUrl(newUrl) was ignored
if newUrl is equal to the URL which is shown in the view already.

The new approach is to reload the view in that method if it is empty, to
make sure that we do not miss that a previously unmounted device has
been re-mounted.

Thanks to Grigoriadis Grigoris for analyzing the root cause of this
issue!


FIXED-IN: 4.11.3

Use the icon size from KItemListStyleOption for the width and height of the
expansion area instead of KIconLoad::SizeSmall.


REVIEW: 113169
FIXED-IN: 4.12

Fix trailing garbage in extracted PDF title


FIXED-IN: 4.11.3
REVIEW: 113138

Fix zoom effect cursor position on initial zoom in.


FIXED-IN: 4.11.3

Fix Bug 321303 - Leave A Note widget: Delivered note presents itself in raw HTML code.

FIXED-IN: 4.11.3

KPropertiesDialog: Do not show the KFileMetaDataWidget

The KFileMetaDataWidget uses Nepomuk1 which is deprecated and is buggy.
It would be better for people to use the Nepomuk2::FileMetaDataWidget.

This mostly only affect Dolphin where users can tag files via the
Information Panel and via File->Properties. This removes the widget in
the File->Properties


REVIEW: 113190

Make the code that removes items from KFileItemModel more robust

When we remove items from the model, we called the function
KFileItemModel::removeItems(const KFileItemList&, RemoveItemsBehavior).
This function then looked up the indexes of the items using the hash
m_items. This is wasteful in the situations when the indexes of the
removed items are known in advance (like when an expanded folder is
collapsed in Details View), and it can cause trouble if one item is
contained in the model multiple times (can happen when searching, and a
file both matches the search and is a child of a folder that matches
the search). Even if expanding folders in the search results list might
not be particularly useful most of the time, it makes sense to make the
model more robust to prevent crashes and other unexpected behavior in
such situations.

This patch makes the following changes to achieve that goal:

* Change the argument of removeItems() from KFileItemList to
KItemRangeList. To make this work, the "look the indexes up in
m_items" code is moved from that function to slotItemsDeleted(). In
the other places where removeItems() is called, the indexes are
calculated directly (which is not more difficult than determining the
removed items as a KFileItemList, if one considers that we needed the
function childItems(KFileItem) for that, which is not needed any more
with this patch).

* Also removeFilteredChildren() takes a KItemRangeList now. Rather than
putting the parent KFileItems into a QSet for O(1) lookup (which
prevents O(N^2) worst case behavior for the entire function), it uses
a QSet<ItemData*> now, which should even be more efficient (hashing a
pointer is cheaper than hashing a KFileItem/KUrl).



FIXED-IN: 4.12.0
REVIEW: 113070

Don't delete a QObject from inside a slot.

If there are any pending calls to that object to be processed deleting
the object can cause them to crash. (such as in KTp's service)

Using deleteLater() is safer.


REVIEW: 113128

Fix Bug 324063 - contact edit dialog has too narrow fields

FIXED-IN: 4.11.3

Don't forget to accept the close event after KDialog::reject().

If we simply ignore the close event, Qt::WA_DeleteOnClose won't
have any effect and the dialog just gets hidden.

Make code robust against event being deleted.

Also fixes a uid() vs instanceIdentifier() issue.

These are the two only possible causes for the crash reported in:

Fix Bug 238574 - show number of notes in systray icon

FIXED-IN: 4.12

Fix Bug 287249 - Import VCF file dialogue is a bit funny

Define minimum size

FIXED-IN: 4.11.3

Fix Bug 310627 - Drag and dropping a link to knotes from Firefox results in invisible text cursor

FIXED-IN: 4.11.3


Keep ktextedit makes work.

Fix Bug 319313 - Select KDE accounts file 'text field' overflows

FIXED-IN: 4.11.3

Fix crash caused by TextChannelWatcherProxyModel

void KTp::TextChannelWatcherProxyModel::onChannelMessagesChanged()
void KTp::TextChannelWatcherProxyModel::onChannelInvalidated()
emit dataChanged() using indexes from the wrong model.

Because it's an QIdentityProxyModel it never causes any harm,
but it's still "wrong" so Qt asserts break in the grouping model.


REVIEW: 113126

Fix apply setidentity (Bug 324532 - kmail filter on sending mails cannot change identity)


FIXED-IN: 4.12

Fix purging to-dos through popup menu.

There was a missing signal.


FIXED-IN: 4.11.3

Fix Bug 297157 - knotes reqires to save edited note twice

FIXED-IN: 4.11.3

Apply thickness parameter to hatching brush

Patch by Salil Kapur

update the slider when scrolling with the wheel

Patch Salil Kapur

Notify the user that JuK is docked on startup.

We show a simple notification if the user starts up JuK and it is set to
dock in the systray on startup (as otherwise the user may wonder why it
didn't open up). Although a "Do not show again" checkbox is not possible
with KNotification, it is possible to remove this additional
notification by going to "Application and System Notifications" in
System Settings to manage "Juk music player" application notifications.

We do *not* show a notification if a session is being restored to avoid
noise, however JuK will show the main window if the user tries to start
it up again after it's already running, so this should be OK.

Thanks to Shubham Chaudhary for the bugfix (you're been marked as the
patch author), and Michael for reporting the bug.

This bugfix adds new strings so it cannot be backported. It will show up
first in KDE SC 4.12.


FIXED-IN:4.12
GUI:

Scale all avatars in contact list tooltips with Qt::SmoothTransformation

Scale all avatars in contact list tooltips with Qt::SmoothTransformation

Without using Qt::SmoothTransformation, scaled images look "grainy".

Fix adiumxtra-protocol-handler to show UI to confirm installation again

Don't install the Adium extra automatically but show a notification to
confirm the installation (as was probably originally intended by the
authors of the code).

Disable the KIO progress notification that results in a non-informative
"Copying [Finished]" notification for the temporary file being created.

REVIEW: 113214

Fix crash caused by TextChannelWatcherProxyModel

void KTp::TextChannelWatcherProxyModel::onChannelMessagesChanged()
void KTp::TextChannelWatcherProxyModel::onChannelInvalidated()
emit dataChanged() using indexes from the wrong model.

Because it's an QIdentityProxyModel it never causes any harm,
but it's still "wrong" so Qt asserts break in the grouping model.


REVIEW: 113126

Fix crash when deleting a "dirty" entry from the wallet


FIXED-IN:4.11

fix keys not being refreshed if a user id is deleted from it

Show clear button in program arguments line edit.


REVIEW: 113186

Implement loading foreign key attributes from XMI

Update documentation window on key navigation

Selecting a different tree item using the keyboard now updates the
documentation window just like what would happen when using the mouse.

The keyboard can also be used to select a different diagram from the tree view.

Implement loading foreign key attributes from XMI




Conflicts:
umbrello/foreignkeyconstraint.cpp

data: add Basque course

Contributed by Alexander Gabilondo

Many thanks!

CCMAIL: Alexander Gabilondo

data: add Workman layout and corresponding course

Contributed by Peter Feigl

Thanks!

kimgio generator: tiled rendering support

REVIEW: 113250

Show the destination file name in the scan progress dialogue (if available)

Tool tips for PNM testing mode and scan size selector

Also change the layout of the two testing mode radio buttons.
Fade the resolution and scan area controls for Virtual Scanner.
Tool tips for Preview and Start Scan buttons.
Trim trailing '.' from SANE-supplied descriptions as tool tips.
Format tool tips as rich text so that they get word wrapped.

kimgio generator: Apply transformations dictated by Exif metadata


FIXED-IN: 4.12.0
REVIEW: 111793

add Gnuplot syntax highlighting file


FIXED-IN: 4.12

Support multiple sessions with separate configuration and cache

Thanks to the work Pali did within the GSoC, Trojita no longer constructs a
QSettings instance from multiple places. It is therefore pretty easy to enforce
a separation of some kind between the concurrent sessions. One use case of this
patch is for situations where it's necessary to access multiple IMAP accounts at
once. It's a pretty sucky hack, but it could serve as a stopgap measure until
proper multiaccounts support is written and integrated.

You can test this now via executing trojita with the --profile option like this:

$ ./trojita --profile work

Of course this patch will cause havoc when support for IPC via DBus is merged
(think mailto: URL handling); that will have to be disabled when a "special
profile" is called. Let's see how well this works for now. Oh, and *please*,
keep the profile name in ASCII; things will break if you don't do so.

For users who do not set a the --profile option, nothing changes and everything
shall work well. It is also safe to run one Trojita instance without any fancy
options and a second one with a custom profile (as long as your profile name
does not match a base64-encoding of some mailbox; please do not do this,
either).

The patch abuses environment variables behind the scenes. I'm not going to put
any more work into this -- the patch is ugly as usual, but this is officially
supposed to be a time-limited hack anyway.

REVIEW: 113150

Make favorite chat room names editable

REVIEW: 113111

FIXED-IN: 0.8.0

DIGEST: Add ability to edit the name of your favourite chat rooms for quick joining

feature: Operations can be split by date

Add more PIN/PUK types for SIM/Modem unlocking

Image transformations can take time, so do them in a background thread.

New class ImageTransform to do this, with operation codes for the
various transforms supported. Combine the multiplicity of slots
in Kooka and the two rotate/mirror functions in KookaView. Also
simplifies ScanGallery.

FileIndexer: Use parameterized queries for inserting the plain text

With Soprano 2.9.4 the virtuoso backend has support for paramterized
queries which consume way less memory when inserting large blobs of
text.

With this the high virtuoso memory usage should decrease to a large
extent.

Refactor now playing plugin

Enable option now behaves as "enable on login", and changes at contact
list will be temporary for this login. And use requested presence to
evaluate the new presence by this plugin.



REVIEW: 113066
FIXED-IN: 0.7