Issue 233

Fix a crash with fancy generator ranges, and add a unit test for that

Git push dialog: Don't segfault at hitting git-svn's pseudo branches

Git pull dialog: Don't segfault on git-svn's pseudo branches, disable ok if no valid branch selected

When saving data objects, sort them first so that they are not saved
before things they depend on. This will not fix old kst files where
the data objects are out of order, but it will prevent bad kst files
from being created in the future.

BUG:
data vectors were reloading the entire file each time they got new data,
rather than shifting them. And shifting didn't use memmove.

These changes should speed up real time significantly.

Provide a proper size hint for the turn instruction pixmaps.

Use FileActionMngr to copy attributes in ImageWindow.
Add code to reset the database rotation flag in case of replacing a preexisting file.

Consolidate code for copying attributes in CollectionScanner and fix copying of tags.

Tags cannot simply be copied at the lowest level: The SQL fails in case tags
are already set, but much more important: internal tags such as history tags
must not be copied. Use TagsCache and ImageInfo facilities to copy tags properly.

Improve auto play behaviour of videos

Use the auto play property of movies that are not started via
the 'page open' action. That's the case for Rendition-based videos
from PDF documents for example.

Set a new Object as parent of BlueZAgent

You can't register a child of a parent exposed with ExportChildObjects, if
you do so the app may crash.

for not making this patch too big I'm using the parent() reference from
the BlueAgent object itself to delete the parent and this will delete
the agent. I will improve it in BlueDevil 2.0

Versions minor to 1.3 are not affected by this bug since they use qApp
as a parent instead of BlueDevilDAemon.

If the device gets paired and no button has been clicked, page is valid

Wiimote is (I think) the first case where SSP is done without having
to confirm any code, this happens because Bluez is doing everything
internally for us (generating the pin for Wiimote). Actually Wiimote
uses legacyPairing internally though bluez is returning false on
legacypairing.

To make this work BlueZ 4.96 is needed

Reset to ascending sort order when changing the column

When changing the sort-column the sort order should always be reset
to 'ascending' to by consistent with the QListView behavior.


FIXED-IN: 4.9.0

Allow Shift+Arrow to be passed into to applications like mc and vim

Note: action shortcuts take precedence of keybindings. So to use
Shift+Arrow in mc/vim, make sure they are not set as action shortcuts.



FIXED-IN: 4.8.3

Never pass null-items to version-control-plugins

Some plugins are not prepared for this and might crash. Although
the additional check in VersionControlObserver::actions() should not
be necessary because of the fix in DolphinView, it is left here
to catch potential other cases where a null-item might get passed
accidently.

Thanks to Ilya for initially patching the Box-plugin which gave
a hint whats going wrong.


FIXED-IN: 4.9.0

This job should be async to do not run into race conditions.


(cherry picked from commit 0ffccae9e54bca8bcfb6be28903c29ed809c06e1)

If the device is a Keyboard, move to the keyboard page.

Thanks to rbelem for lettimg me steal his bt keyboard at UDS xD

Only empty the Overlay if there are powered devices.

tired of launchers not working because there isn't a .desktop file

we KNOW when there is an exec that we can run it; for the broken apps out there
that do not have a .desktop file, are not in the fix-up lists AND can not be run
from their executable .. screw 'em. i'd rather have the real world work even if
that means edge case applications written by idiots fail.

fix quick unmaximization position


REVIEW: 104702

Do not create Deleted on Workspace Shutdown

When the Workspace is shutting down the compositor is torn down
before Clients and Unmanaged are released. This means that there
is no need to create the Deleted windows.

Furthermore creating the Deleted manipulates the stacking_order
while Workspace dtor loops over this list to release all clients.
This may cause crashes.


FIXED-IN: 4.9.0
REVIEW: 104690

check for an absolute path in the correct way.

When Esc is pressed, clear the selection and cancel the keyboard search

This patch restores the Dolphin 1.x behaviour that Esc clears the
selection (Dolphin 2.0 did not have an easy way to clear the selection
using the keyboard so far). Moreover, Esc now also cancels the current
keyboard search. This fixes the problem that the only way to cancel the
search is to wait until the timeout has expired.



(cherry picked from commit d3b1312d3edc615b03fb17c1a5fa62ccb61cdabd)

Toolbutton: Fix layout issues

- Improve spacing between icon and text
- Do not enforce a minimum width

Having a minimum width does not make much sense for a ToolButton:
1. It should aim at using the minimum amount of horizontal space when
used in a ToolBar.
2. It looks unbalanced when used with an icon because the content is
flushed to the left, leaving a large amount of white-space on the right.

REVIEW: 104735

Details mode: Fix wrong required column-width calculation


FIXED-IN: 4.9.0

Cleanly remove all ObexSession


(cherry picked from commit fa722d611c661509d8fd819f484cc748d060e8b3)

When the current item is removed, make -1 the current index temporarily

This fixes two problems:
1. KItemListKeyboardSearchManger can cancel the current search when a
new folder is opened (note that this action removes the current item
from the view).
2. The view can underline the new current item (which is the item that
used to be below the removed item). Note that this did not work
before because the view did not receive a currentChanged() signal in
this case and therefore did not update the "current item" status of
the new current item.



REVIEW: 104709
(cherry picked from commit 68ce395a192362969783615e50a8004d3029eb7e)

Fix Bug 298861 - Kontact crashes when switching to summary.

Todo can be null. Need to verify it

FIXED-IN: 4.8.4

Fix Bug 128365 - edit recent addresses: add sorting order and modifier

selection
FIXED-IN: 4.9

Don't update event UID after it's created

Untill now we tried to update UID of a newly created event after we received
the UID from Google, but KOrganizer didn't cope well with this.

This patch removes the IteModifyJob which was updating the UID. We rather
keep the KOrganizer-generated UID and we only change it in the payload before
serializing it and sending it to Google (for instance when we want to modify
the event). This makes KOrganizer happy.


FIXED-IN: 0.3.1

Don't remove resources before updating them.

We can simply overwrite the properties. Before also PIMO things were removed because they are marked as subresource of the item,
(which is correct), but therefore linked resources were lost on every update.
This patch resolves that, with the drawback that plugins need to take care of outdated subresourcecs themselves.

Fix Bug 298181 - add title of messagesright window

FIXED-IN: 4.8.3

Fixes: ensure same order of units in whole Calligra UI, have ruler and page layout dialog update to change of document's unit type

also tune KoUnit API a little:
* rename KoUnit::unit(...) to KoUnit::fromSymbol(...) <- more Qt'ish
* turn static KoUnit::unitName(KoUnit) into member method KoUnit::symbol() <- as typical use-case is on existing KoUnit instance, also shorter and more OOed
* rename KoUnit::Unit to KoUnit::Type <- "type" feels a better term here
* added KoUnit::type() and KoUnit::setFactor(...) <- useful in a few places
* remove KoUnit::unitDescription(...) from API <- not used outside
* rename KoUnit::PixelVisibility to KoUnit:: ListOption <- more general, some might want to add other flags like HideImperial

And adds unit tests for the touched KoUnit API

Thanks to boemann and Cyrille for making it compile with Qt 4.7 as well

REVIEW: 104626

Bugdet reports: Skip budget account if periods for the account
have not been loaded

REVIEW:104766

fix crash on opening pdf with special characters in the path

Handle the situation where the ng iterators get an empty stretch

Assert in developer mode so we know where to add the check for emptiness
before creating the iterator; make the iterator check at least a pixel
in release mode.

We cannot re-instate the old isDone() api because that would impair
performance too much:

13:26:45 < boud> CyrilleB: do you remember why the NG iterators don't have an isDone() check so the loop always checks at least once? (with do {} while
(it-NextPixel())
13:45:34 < CyrilleB> boud: performance
13:46:13 < CyrilleB> boud: especially since in 99.99% of the case, one does have something to iterate over
13:50:34 < boud> aah
13:50:49 < boud> because it's the reason of the crash in KisSelection::invert
13:52:29 < boud> maybe if the rect is empty we should make it have a width/height of one so there's always one pixel read in all cases
13:53:04 < CyrilleB> or rather have an assert if you try to create an iterator with a null width/height
13:53:20 < CyrilleB> (or both, if you want to ensure good behaviour for users)
it goes wrong
13:53:35 < CyrilleB> s/good behaviour/non crashing behaviour/

To delete a report from the list, do not open the report.
This helps when trying to remove a faulty report, which cannot be opened
correctly.

REVIEW:104763

Prevent kmixctrl running with PulseAudio.

kmixctrl should not save/restore volumes when PulseAudio is used, but
our event loop detection was such that the PulseAudio mixer would refuse
to run as kmixctrl was marked as a non-GUI program

IpodCollection: rework iPhone identification to prevent false-positives

...false-positives that would otherwise appear once the change [1] is
merged into kdelibs. (should happen for 4.8.4)

[1] https://git.reviewboard.kde.org/r/103028/

Clean up url management

- Let ftp url(s) work again
- Do NOT check url twice via KUriFilter (removed MainWindow::filterUrl)
- ensure we are lowering just http url hosts
- mailto via urlbar handling

Fix opensearch add shortcut handling

(at least on every site tested, but opensearch.org itself. Broken?)

Do not show "closed tabs" page after crash when directly opening an URL

DSL: do not send empty service field to NetworkManager or it will not
create the connection.


FIXED-IN: 0.9.0.2

Make use of DialogEngine

Make sessions model class with a property to show/hide the entry "Show Last Used Session"

Updated the classic theme to use this. Userbar theme does not have this entry.

In the classic theme highlight select password field content and force focus on wrong passwords

Toggle showing of rating on Viewer InfoBox

Added possibility to toggle on or off the viewing of rating in InfoBox
in Viewer.

Key Bindings for Switching between Windows of current Application

Additional TabBox Mode which allows to switch between all open
windows of the current selected application. By default Alt+tilde
is used which is on qwerty just one key above tab. For non-qwerty
layouts the shortcut is unfortunately not convenient.

REVIEW: 104730

FIXED-IN: 4.9.0

Provide additional default groups for the Places Panel

If Nepomuk is enabled, it is now possible to easily search for
some most common queries by having additional groups.

Update the view when changing the directory using 'cd' in the terminal

Thanks to Jekyll Wu for helping to implement this feature!


FIXED-IN: 4.9.0

Add two profile options for the copy & paste behavior of mouse

* support "automatially copy selected text into clipboard"
* mouse middle button can be customized to paste from selectio or
clipboard

The first feature might look like a unnecessary duplicate of the
"synchronize selection and clipboard" feature of Klipper. However, that
klipper feature is generally problematic, while doing it only in Konsole
is generally useful and harmless.


FIXED-IN: 4.9.0
REVIEW: 103861
DIGEST:

Allow QML items to be associated with operations in Plasma::Service

QML items derive from QGraphicsObject, but not QGraphicsWidget. We
actually only need QGraphicsObject (for the enabled property).

This allows calling (dis)associateWidget from QML (eg: in javascript
plasmoids) and passing in a QML item.

Now we can specify what we want to backup/restore

Fix Bug 298245 - google like search behaviour in quick search

now we can search in search line as separate string. So it will improve
search.

FIXED-IN: 4.9

Make the Video shape loading and saving work, even to and from
LO/OO Impress (although impress and multimedia atm is useless)

also improve the painting of the actual shape with a nice icon

Google Sync Bookmarks in rekonq!

- Added UI bits to configure Google Bookamrks Sync
- Implemented sync, 2-way merging local and remote bk
- Remember to not update local history while remotely
adding bookmarks
- Encode parameters of add bookmarks request while posting data.
- (adjam) fix compile warnings

This is the squashed commit of Situ's work on Google Bookmarks Sync
SQUASHED BY: adjam ;)

REVIEW: 104673
DIGEST: Google Bookmarks Sync
REVIEWED BY: adjam

more possibilities how to instantiate Hand()

so we can give Hand a pointer to Game.
This weakens the very strict API where Hand knows nil about Game
but that would probably be difficult to maintain longterm anyway.
In practice, the API is still the same - we do not have to define
a game but if we do we get better debug log messages (next commit)

framework for defining AI filters in rules

the standard AI filters would never be able to reach those limit
hands which do not have the standard pattern.
One empty example is included.

Only create decoration once during Client::manage

Client::manage calls setupCompositing on the not
yet fully managed client before it wants to create
the window decoration. This means the decoration is
created twice and maybe even if not needed (noBorder
not yet set).

If the Client is not yet managed setupCompositing
will not update the decoration - it will be handled
by manage later on.

Furthermore if compositing is not active, setup
compositing does not have to perform any action.

REVIEW: 104758

Improve ApplicationGridDelegate in many ways

- Reduce redundant constructions and unneeded id's
- only set the description component when needed
- remove conflicting width definitions: anchors.left/right
vs. width
- Don't show any rating if there are no reviews.

Items are shown and got their layout on the board. For now, the whole tile group is not centered! Also the size changed to fit on the given space of the GameWidget.