Issue 235

Prevent different threads to modify the KDirWatcher instance at the same time

When adding plots from the console, use the resulting expression instead of the input

This way we don't move declarations around.

Handle download of map regions for themes without texture layers.

Fixes crash when opening the download region dialog when e.g. the plain
map is opened. Related: Crash when attempting to download an empty set.
Crash when switching to a map theme without texture layers after the
download region dialog was opened at least once before.

Signal toggled() to apply state will not be emitted when the per default unchecked checkbox remains unchecked.
Call explicitly.

Fix crash due to infinite recursion

Guilty commit: 0e0318e5fed517f50a2d356fda88fa3ec3c7d90f "use dedicated namespace"

CCMAIL: Gilles Caulier

Ensure ScreenTransformation matrix is reset after rendering cube

Cube effect modifies the ScreenTransformation matrix. While each
window resets the matrix on a global level it's not reset. That
means if the last rendered window had the screen rotated the
ScreenTransformation keeps the rotation even after the cube
ended. This causes a rotated window in e.g. TaskbarThumbnails.

REVIEW: 104918

Do not clear selection after pasting.

The old behavior has a long history, initially introduced by commit
1e3c3e27 more than 10 years ago. I fail to understand its intention
(some safeguard?), and I think keeping the selection is more useful in
work flow.

Keep it in master branch for now. No backporting.

Reimplement search-filters



FIXED-IN: 4.9.0

Fix animation of scaled GIF images.

Firstly, implement the AnimProvider::clone function in the GIF loader.
This enables the creation of scaled AnimationProviders.

Secondly fix the ImagePainter so that it uses the set size instead of
the size of the base (unscaled) image.

Lastly fix RenderImage::updatePixmap so that it takes scale into
account when deciding which rect to repaint.

handle m_currentBuildQuadsIterator validation independent form painting pass

issue is that the effect chain may (will) change between two paint passes and buildQuads is called outside the paint pass
(shadows / decos whatever changes) so that the iterator may dangle


FIXED-IN: 4.9
REVIEW: 104881

Exclude minimized and windows not on current desktop from fade desktop


FIXED-IN: 4.9.0

Reprompt the user with a proxy authentication dialog even when the last proxy
authentication dialog was cancelled.


FIXED-IN: 4.8.4

Delete cached icons in-memory when changing icon theme.

This should hopefully fix bug 294044 ("Dolphin won't accept custom
icons"), although I can only partially reproduce it. I was able to
verify that icons left in the KSharedDataCache for running processes are
preferred without checking the new theme, which this patch fixes.


FIXED-IN:4.8.4

Fix connect() when checking radio buttons

FIXED-IN: 4.8.4

MERGE: 4.8

Fix Bug 299344 - Original sender is not kept when using "Send Again"

FIXED-IN: 4.8.4

Fix Bug 297134 - Filter action "mark as read" doesn't mark as read

anymore since 4.8.1
FIXED-IN: 4.8.4

Use a safer way to set the minimum width of the account combo.

Using QFontMetrics::maxWidth() was a bad idea because some fonts could
contain really wide characters (with a very low probability of appearing
in the account names) - see the bug that is fixed.

Instead of the width of the widest character in the font use the width of
the 'W' character which will produce a much more predictable result.

Even the Qt implementation uses the width of a know character ('x') to
compute a kind of minimum size.

Fix issues with recovering autosave files

1) X-KDE-NativeExtension can be empty, in that case take mime->mainExtension.

Otherwise our autosave files are called .autosave, and that's not something
the filter's mime detection can handle

2) don't overwrite autosave files when running multiple unnamed modified
documents

3) tag the autosave files with the pid and use dbus to check which processes
that have autosave files are still active; don't offer to recover those

4) show the autosave files in a nice dialog with the ora/odf thumbnail
as a preview, and offer to recover them all.

REVIEW:104892

Do not delete some static stuff on exit

Depending on the order of deletion and other variables, these delete
calls might cause a crash on exit or not. And there's actually no
memory leak, since this is all static stuff that's present for the
lifetime of the application.

In this case, it's better to just quit, than explicitly free all memory.
The bytes are gone with the process, and there isn't stuff that needs
to be written on quit here.

Don't add colors to the history while erasing



We probably have multiple color history implementations in Krita though,
those really should be squashed together.

Strip white spaces when importing kpr files

Seems this breakage went unnoticed when the stripping of whitespaces was removed.
Fix the problem by stripping the white spaces when importing the old documents
which always stripped the white spaces.

Ok to backport?

Do not resize the docker column when collapsing the docker

We keep track of the pre-collapsed titlebar widget width now and use
that in the sizeHint() method.

Remove context view video applet

Unmaintained and broken for quite some time.


FIXED-IN:2.6

AudioCdCollection: don't create a reference out of temporary var (crashfix)

There's following code in AudioCdCollection::updateProxyTracks():
const QString &urlString = url.url().remove( "audiocd:/" );

Which I think is wrong, because url.url() returns a temporary QString
and QString::remove() returns a reference to itself. However, the
temporary is AFAICS destroyed as soon as this line ends. This IMO
results in urlString being an invalid reference.

I'm not really sure about this, I haven't read C++ spec, but given many
users report crash on the following line, this could be the culprit.
I was never able to reproduce the bug, so I'm shooting blindly.

Reporters, please test reproducibility with current git and reopen if
this is not fixed.


FIXED-IN: 2.6
DIGEST: fix grave crash

Daap::Reader: Don't use QList::[ on unreliable input, use ::value()

This should fix bug 280774, but I don't use DAAP collection, do I'm
shooting blindly.

BUGFIXES:
* Fix crash related to DAAP collection.

Reporters, if this doesn't solve your bug, please reopen.


FIXED-IN: 2.6
DIGEST: bugfix

Collection browser: don't add duplicate tracks in mimeData

That could have happened when you selected e.g. both album and a track
from that album -> the track would be added twice.

The implemented algorithm is done at QModelIntex level and is O(n)
where n is the number of selected tracks so this should be okay.

There is an unrelated bug with the Various Artists item that behaves
as entire collection when dragged. This will be fixed soon.

BUGFIXES:
* Don't add duplicates to playlist when dragging from collection


FIXED-IN: 2.6
DIGEST: Bugfix: Amarok now doesn't add unwanted duplicates to playlist

Move WriteTagsJob out of IpodCollection and use it everywhere

CHANGES:
* Album cover images are written in background to prevent freezes


FIXED-IN: 2.6
DIGEST: Amarok now writes back covers in background to prevent freezes

Hopefully fix crash in ContextView on startup

It is strange that simple initialization to zero fixes the problem, but
it fixes the crash for at least one reporter.

Reporters, please rebuild Amarok from current git and should loudly if
you can still reproduce the bug.

BUGFIXES:
* Fix crash on startup related to ContextView


FIXED-IN: 2.6
DIGEST: Amarok crashfix

Fix hidden wireless network creation.


FIXED-IN: 0.9.0.2

Fix null pointer crash

Reviewed-by: David Edmundson

Kopete export contacts dialog - Fix checking if contact was already exported

Before this commit Kopete show that all contacts (with contacts already
linked into addressbook) were not in addressbook.

move left modify action in favorites to be the update one

Also implement ThumbUpdater class to easily manage thumb updates

Make the delegate overlays code signicitantly simpler, fixes crash in the process

The overlays were in several classes, all deriving one from another. This commit merges several of the classes into one and removes unused and/or useless methods, thus simplifying the code and making it more readable, effective and easier to maintain.

Reviewed-by: David Edmundson
REVIEW: 104887

Force termination of Kile when the main window is closed.

Under some circumstances the Kile process would still remain in memory otherwise.

The basic featues of CommandEntry work

CommandEntry is usable, but not everything works as of yet:
- SyntaxHighlighting is crippled. It does not work in Maxima, but
in Qalculate some things are highlighted.
- Completion does not work, because Tabs are not recognised.
- The WorksheetTextItems do not look nice: They have a wierd border,
and the cursor is not shown.
- WorksheetTextItems are also too small, they should fill the whole
line.
- Cursor movement does not work for additional information blocks.
But I am not sure whether one should be able to navigate out of
those, as any modification outside should abort the evaluation
and destroy the additional information.
- There most certainly is more.
Other Entries can not yet be added to the Worksheet.

Search for videos without thumbnails on startup

Make sure the translucent background is available whenever possible

Now the Qt::WA_TranslucentBackground attribute is always set for the
mainwindow of konsole(regardless of whether compositing is available
when the window is created), just like in konsolepart.

I'm not 100% sure this approach is problem-free, but it works for me so
far under various environments and conditions. Do not hesihate to revert
it if you notice it causes big trouble.

Get Hot New Stuff support for KWin Scripts

Button currently not yet shown as the category on kde-look is
still missing. This means the code is also completely untested.

REVIEW: 104877

Allow moving items to trash also for non-local files

See bug 188032 for an ongoing discussion. In case if this behavior
is really not wanted by users (the feedback on bugs.kde.org will
show), the review-request
https://git.reviewboard.kde.org/r/104915/ should be merged too
(after adjusting some parts of the code).

Allow to remember view-properties for the search-mode

Per default Dolphin remembers the view-properties for each search-URL
individually. However when being in the search-mode, where the
search-URL is different each time by definition, it should also
be possible for the user to adjust the view-properties. The DolphinView
is not aware about any "search-mode", so an interface has been added
to provide a "context" for storing the view-properties. The context
will be adjusted by the DolphinViewContainer in case if the search-mode
is active.

Screen Edge bindings for Scripts and Scripted Effects

ScreenEdge is changed to emit a signal whenever a screen edge
got activated without an action or effect taking care of it.

A Script can reserve one to many callbacks for an edge and the
callback get's triggered whenever the signal is emitted. On
deconstruction of the Script the edge is unreserved again.


FIXED-IN: 4.9.0
REVIEW: 104904

Allow to remember view-properties for search-results

Until now it was only possible to adjust the view-properties for
searching in general. Now the view-properties can be adjusted
dependent on the query (e.g. images, documents, ...).

Asserts for KWin scripts

Further debugging functionality for KWin scripts. Added assert
methods validate the to be tested parameter and throw a script
error if the value is not valid.

Following methods are available:
* assert(value)
* assertTrue(boolean)
* assertFalse(boolean)
* assertEquals(expected, actual)
* assertNull(nullValue)
* assertNotNull(notNullValue)

All methods take an additional optional parameter which is used
as the error message if provided.

Methods to validate the number of arguments and types of the
parameters are added and throw syntax or type errors.

REVIEW: 104870

Save size of message/mime splitter on change

This way when you change the mime size splitter and select a new message in the list
the expect thing happens (i.e. the sizes of the message/mime widgets remain the same)
(cherry picked from commit 1c448b1dbdceeb5a456ba7a4431551792a43ce1c)

REVIEW: 104916

Add abstract Calendar (Evolution has calendar so we can import it)

Fix Bug 259784 - It would be nice to be able to print only highlighted

parts of an email

FIXED-IN: 4.9

Bibliography database user interface.
Loads citation database from sqlite file.

Add a floating message class

This can be used to show a transient message that fades after some
time. It's used here to show the user the shortcut for getting out
of the full-screen mode.

Better CSV and TXT exports of tables grouped by a column

Make author information configurable on a settings level.

The Document Information dialog's Author tab becomes read only and just shows who
last saved the document.

The author information is shared across all calligra apps, and can hold any number of
author profiles. The user creates and deletes author profiles through the settings dialog.

The profiles are stored in the calligra wide config.

In the settings menu a submenu should be added where you can choose the active profile.

The active profile is stored in the application's config.



It also solves the bug (no number) that save cycles number was never incremented

add lyricswidget

Don't stop playing Music or Videos while viewing Pictures

Amazon store: popupdropper icons

Thanks again Fabi for the great work.

Hopefully I added them correctly to the popup dropper, but things seem
to work fine here. I also just realized that one icon is still missing:
load details.

When the lid is closed, suspend the computer

REVIEW:104914

Add Retry button to one of the extraction dialog's prompts.

As written by the patch author:
With this patch, there will be a working Retry button on the dialog
box made in ExtractionDialog::accept().

This implements an old TODO item written by Harald a few years ago
(commit f761cbd164c1015100d810ec3c02a60b78969891), and users are now
able to retry when an extraction into a subdirectory will overwrite an
existing location.

Submitted by: Nathan Mills
REVIEW: 104869

Set a configuration KConfig file to provide the app some data.

The idea is that the packager should be able to provide some data
to the application to specify some distro specific sources.

At the moment there are 3 of them: it lets you choose what debian
screenshots instance you'd like to pull your information from, also
the ratings and reviews server and the featured apps.

Add read support for cab archives.

Configure data types in separate dialog.

This solves the mixture of settings with global effect (to all
data elements of a type) with settings that only affect the current
data element. All configurations are now possible by calling the
data type properties menu.

Places Panel: Implement eject and teardown actions

Further fixes:
- Add/remove item when device has been added/removed
- Update emblem if the accessibility-state has been changed

Add cmdline options for the visibility of menubar and tabbar

Since the menubar and tarbar settings are now global instead of per
profile , those cmdline options are useful to allow users to override
the defautl behavior.

The --hide-menubar and --hide-tabbar options can be used together
to simulate the miminal interface of xterm. I'm not sure whether
--show-menubar and --show-tabbar will be widely used , but add them at
the moment for the sake of completeness.

Don't find x positions unless we need to, this should give a factor 2 speed up in text rendering

Do layout too much. On big documents that can disasterous

In fact it's annoying we have to layout for this at all. We should store
this info in KoTextBlockData, and do our own drawing. It will be both more beautyful
and be prepared for visualizing both spelling and grammar and similar.

Only clear pixels inside a paint device

On clearing pixels, only clear the pixels that actually have content, so
use the paint device's default bounds object to get the intersection
between the selection and the paint device.

Improve memory consumption of the screenshots in the grid delegate

Set a source size for the screenshots so that it doesn't keep
the full size version in memory.

- Replaced all references of 'search' with 'web shortcuts' since web shortcuts
are not only for doing search.

- Replaced all the WhatsThis help text with equivalent ToolTip text.

Note that this change only renames related config options, variables and GUI text.
In the future eveything, including file and class names, should be renamed to
reflect the current functionality of this plugin.


FIXED-IN: 4.9.0
GUI:

- Moved the checkbox used to select preferred shortcuts into its own column to
prevent user confusion.

- Allow the shortcut list to be filtered using the shortcut texts.



FIXED-IN: 4.9.0
REVIEW: 104900
GUI: New translatable text

Revert commit 462a06ea as it causes many regressions. Use KIO::synchronousRun
to fulfill synchrounous XmlHttpRequest instead.

Unofrtunately this means that the fix for the crash reported in bug# 287778
is reverted until we can find a solution for the side effects of using nested
event loops.





FIXED-IN: 4.8.4

Have a border around the canvas - it's the widgetstyle's responsibility to make sure
it looks nice

Downloads search