Issue 212

4th December 2011 by KDE Commit-Digest Team

Contributors

Vladislav Blanton
Danny Allen
Marta Rybczynska

This Week...

Embedded mode for TabBox in KWin. Reimplemented name filtering in Dolphin. Initial support for resource events in read-only kparts in kdelibs. Calligra updates include improved opening of .kexis files and creating new app instances and main window UI changes in Kexi; support for filled radar charts, updated document handling and optimization of reverse iterator and ODF saving. Improved download management and shortcut+slot to open downloads page in rekonq. KDevelop supports configuration of some environment variables for CMake jobs. Export of non local files in Skrooge.

Statistics

Commits 2450 by 198 developers
Open Bugs 22755
Open Wishes 17286
Bugs Opened 460 in the last 7 days
Bugs Closed 1016 in the last 7 days

Commit Summary

Module Commits
/trunk/l10n-kde4
262
 
/branches/stable
86
 
/trunk/l10n-support
43
 
/trunk/KDE
37
 
/branches/work
22
 
/trunk/www
21
 
/trunk/playground
18
 
/plasma/generic
16
 
/kdegames/assertions.py
15
 
/src/oxygenqtsettings.cpp
14
 
Files Developer Commits
270
 
Marco Martin
93
 
273
 
Aaron J. Seigo
91
 
174
 
Lamarque Souza
65
 
180
 
Laurent Montel
61
 
150
 
Laszlo Papp
56
 
126
 
Albert Astals Cid
42
 
108
 
Sebastian Sauer
38
 
105
 
Burkhard Lück
37
 
108
 
Yuri Chornoivan
36
 
84
 
David Faure
29
 

Internationalization (i18n) Status

Bug Killers

Person Bugs Closed
Aaron J. Seigo
197
 
Myriam Schweingruber
103
 
Jekyll Wu
95
 
Leonardo Finetti
57
 
gregor freenet de
38
 
Dawit Alemayehu
32
 
Lamarque Souza
30
 
Anne-Marie Mahfouf
27
 
Andrea Diamantini
27
 
Christophe Giboudeaux
27
 

Commit Countries

Commit Demographics

Sex

Age

Contents

  Bug Fixes Features Optimization Security Other

Accessibility

     

Development Tools

[] []    

Educational

     

Graphics

[] []    

KDE Base

[] [] []   

KDE-PIM

[] [] []   

Office

[] [] []   []

Konqueror

     

Multimedia

[] [] []   

Networking Tools

[] [] []   

User Interface

     

Utilities

  []    

Games

     

Other

[] []    

There are 118 selections this week

Bug Fixes

Development Tools

Nick Shaforostoff committed changes in /trunk/KDE/kdesdk/lokalize/src:

make overwriting existing notes impossible:
edit existing note instead, if multiple notes are not supported
(this is the case for PO)

Andreas Pakulat committed changes in [kdevplatform] util/environmentselectionwidget.cpp:

Fix storing/restoring of environment settings

This works around a change in kdelibs that was done to fix
KConfigDialogManager's behaviour with Qt4.8 and normal Q/KComboBox. With
that change the user-property is not considered anymore for Q/KComboBox
and instead the hardcoded index/text-handling part is done. That breaks
our own KComboBox subclass since we relied on the fact that our own
user-property would override that hardcoded stuff.

After looking at the kconfigdialogmanager sources I noticed that it
supports a special property "kcfg_property" which can point to a
property-name and then is considered first as the property to store
into the config file. With that we can again restore the behaviour of
storing the currentProfile to the config file.

This will work on all kdelibs back to sometime 2009, which covers KDE
4.5 and later at least, which is enough since the code requires 4.6 anyway.

Aleix Pol Gonzalez committed changes in [kdevelop] projectmanagers/cmake/cmakemanager.cpp:

Fix integration with custom output paths in CMake.

Graphics

Albert Astals Cid committed changes in [okular] /:

Remove the need for the hash so we can map back from words to characters

This is needed because sometimes the hash had collissions and make it impossible to know which character we had to put back
Now we just keep the word and the characters together in the same class and it is much easier to correlate them
Also the code gets much more simplified and less new/delete are needed

This fixes the crash in 287138

I am still concerted that we use the page width and height in TextPagePrivate::correctTextOrder, but that should not cause crashes, at most some misplacements of very small text

Albert Astals Cid committed changes in [okular] generators/poppler/generator_pdf.cpp:

Properly parse "characters" formed by two QChars

This fixes the crash at 287138, it does still not really fix the flaw in the algorithm in TextPagePrivate::breakWordIntoCharacters
when there are two strings with the same text and the same bounding rectangle in the same page, but i still don't know how to fix that problem properly,
so i've fixed a bug that was generating the situation for files that actually do not have it

KDE Base

Philipp Knechtges committed changes in [kde-workspace] /:

kwin: bugfix nouveau GLES + loose texture bindings in kwin_gles

There seems to be a problem with nouveau GLES if you want to create an
EGLImageKHR more than once in a frame for the same pixmap. This patch
circumvents the problem in the way that it implements tfp the same way
as the mesa example in
mesa/demos/src/egl/opengles1/texture_from_pixmap.c does it. A nice
side effect of this is that it also avoids the overhead of recreating
the texture for every damaged window.

REVIEW: 103303

Diffs: 1, 2, 3, 4 Revision 6e44b07...
Aaron J. Seigo committed changes in [kdelibs/frameworks] plasma/applet.cpp:

constrained square makes sense for vertical but not horizontal panels

Aaron J. Seigo committed changes in [kdelibs/KDE/4.7] plasma/packagestructure.cpp:

catch another possible mimetype identification

Aaron J. Seigo committed changes in [kdelibs/frameworks] plasma/tooltipmanager.cpp:

use the default corona in case the widget doesn't provide one

we need a different solution for libplasma2, however.

Aaron J. Seigo committed changes in [kdelibs/frameworks] kutils/kpluginselector.cpp:

Fix regression induced (but likely not caused) by commit e91e5fed6b1aad365e12e919f430c3e8147552d3

Commiting on behalf of Jaime Torres.

Ratio:
On k-c-d I suggested this would be another ouotcome of bug #213068 and Jaime could confirm this and the
scrollbar policy workaround to work.
He suggested to fix it by "setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);" instead (since presumingly the
pluginselector is always top-down oriented)
I tested that variant as well and could not reproduce the issue.

Since Jaime has no acces to his development environment for today he asked me to either revert his commit or
apply the hotfix working around bug #213068 as mentioned above.

Because i could not spot any bug in his commit i'll go with the hotfix to prevent a regression.
Resolving the actual recursion bug is an outstanding issue.

Aaron J. Seigo committed changes in [kdelibs/frameworks] plasma/applet.cpp:

set the parent of the config skeleton object to be the dialog

Gregor Tätzner committed changes in [kde-baseapps] plasma/applets/folderview/folderview.cpp:

Folder View pointed at Trash doesn't update the context menu properly

enable empty_trash action in contextualActions when the url is pointing
to trash:/. contextualActions checks to see if the url protocol is trash
before adding the empty_trash action, but does not check to see if it
should be enabled or not.


REVIEW: 103282

David Faure committed changes in [kde-runtime] kuiserver/jobview.cpp:

Don't assert on dbus communication, just print the error and return.

I could reproduce this by stopping plasma in gdb after starting a job.
The dbus call to plasma would time out, and we'd get a dbus error here.

Xuetian Weng committed changes in [libstreamanalyzer] plugins/lineplugins/cpplineanalyzer.cpp:

we need a useful new property or new DataObject type to describe a C
header which does not have an absolute path.
Comment it out for now.


REVIEW: 103258

Thomas Lübking committed changes in [kde-workspace] kwin/libkwineffects/kwinglutils.cpp:

deny glsl by limited NPOT, not GLSL


REVIEW: 103283

Aaron J. Seigo committed changes in [kde-workspace] plasma/generic/dataengines/tasks/tasksengine.cpp:

workaround qt ref counting bug seen triggered often in libtaskmanager

not patching libplasma as i'd like to keep this mess limited to taskmanager related code
rather than burdoning all DataEngines with it. this requires today's KDE/4.7 libplasma to
work as expected: sourceRemoved was not being emitted for direct-deleted containers.


REVIEW:103247

Dawit Alemayehu committed changes in [kwebkitpart] src/kwebkitpart_ext.cpp:

Correctly set the host that is going to be blocked by the ad filter.

Vitor Boschi da Silva committed changes in [kdeplasma-addons/mediaplayer-4.9] /mediaplayer:

Display media title on Mediaplayer applet

it uses metadata when available and fallback to file name.

REVIEW:103315

Aaron J. Seigo committed changes in [kde-workspace] libs/plasmagenericshell/scripting/appinterface.cpp:
Aaron J. Seigo committed changes in [kdelibs/frameworks] plasma/view.cpp:

containmentType() .. NOT type()!

and now we know the source of the "panel covers the entire screen" bug

Aaron J. Seigo committed changes in [kdelibs/frameworks] /:

Fix Date Display in dolphin info panel

Date localization need to be done within kfilemetadatareader process,
and need to use localtime instead of UTC time.

REVIEW: 103300

Aaron J. Seigo committed changes in [kdelibs/frameworks] plasma/private/tooltip.cpp:

get rid of the text halos

they are (relatively) expensive and should be unnecessary now:
we have blur, it works well, and when we don't blur we use different backgrounds.

one more "take one thing off" while i'm futzing around in tooltips

Aaron J. Seigo committed changes in [kdeplasma-addons] /frame:

don't hold on to the job pointer, it gets passed back to us in the completion slot

Aaron J. Seigo committed changes in [kdeplasma-addons] applets/rssnow/news.cpp:

don't resize the news widget

Jekyll Wu committed changes in [konsole] /:

Revert "Ensure %w and %# are also expaned in the 'Copy input' dialog"

That fix is not good enough because it makes '%w' being updated only
after keystroke instead of immediately . The issue of updating title is
actually a little more complicated than I have understood.

Revert it because the regression should influence more users than the
fixed problem does. Need to spend more time to find a better fix.




This reverts commit 1c1981501b6e3bcd433840bf36f7851f06772322.

Aaron J. Seigo committed changes in [kde-baseapps] plasma/applets/folderview/folderview.cpp:

mime ptr may be null apparently

Aaron J. Seigo committed changes in [kde-workspace] libs/plasmaclock/clockapplet.cpp:
Dawit Alemayehu committed changes in [kde-runtime] kpasswdserver/kpasswdserver.cpp:

To avoid user confusion, hide the "Remember password" checkbox whenever the
wallet is disabled.

Note that this does not affect credentials being kept for the duration of
the application(s) that set them or are currently using them. It only deals
with persistent caching of password in KDE wallet.


FIXED-IN: 4.7.4

(cherry picked from commit da6f4c128d32de782d4e70e11a083f4e20753242)

Martin Klapetek committed changes in [kde-workspace] /:

Fix regression in Klipper, which broke the "Show at mouse position" feature back in 4.5.

Committing this with i18n approval.

REVIEW: 103196

Aaron J. Seigo committed changes in [kde-workspace] plasma/desktop/applets/tasks/taskgroupitem.cpp:

sync the dialog to the graphics widget before moving it

David Faure committed changes in [kwebkitpart] src/networkaccessmanager.cpp:

Handle helper protocols here, so that links to ssh://host work

(after the security warning).
Helper protocols have to run an app, there is no kioslave associated
with them. KRun has the code for it.

Colin Guthrie committed changes in [kde-runtime] phonon/kcm/audiosetup.cpp:

phonon: Allow the audio setup GUI to work second time round.

This basically clears things out properly such that the second invocation populates
it all again with fresh data.

KDE-PIM

Allen Winter committed changes in [kdepimlibs] akonadi/collectionstatisticsdelegate.cpp:

in getCountRecursive() don't assert on invalid collections; instead,
just ignore invalid collections. this is because collections could be
deleted in the meantime and deleted collections are invalid collections.


FIXED-IN: 4.7.4
MERGE: 4.7

Sergio Luis Martins committed changes in [kdepim] korganizer/printing/calprintpluginbase.cpp:

Fix crash when printing.

Don't ignore the return value of erase and/or don't assume
vector.end() never changes.

Torgny Nyblom committed changes in [kdepim-runtime] agents/maildispatcher/sendjob.cpp:

Make sure that the Bcc header is removed

When sending a mail make sure that the Bcc header is removed before
sending the mail.


REVIEW: 103308

Laurent Montel committed changes in [kdepim] /:

Fix Bug 287779 - Copy Email then Paste As Attachment does not work as expected


FIXED-IN: 4.7.4

Allen Winter committed changes in [kdepimlibs] kmime/kmime_util.cpp:

a bunch of little things found by valgrind to help prevent crashes:
- init memory
- check return codes
- check for negative length values passed to QByteArray::mid()

MERGE: 4.7

Laurent Montel committed changes in [kdepim] /:

Fix Bug 288007 - kmail filter import broken

now we can import old filter. We can export/import filter.
FIXED-IN: 4.8

Allen Winter committed changes in [kdepim-runtime] resources/mixedmaildir/retrieveitemsjob.cpp:

in Private::storeListResult(), don't try to create a delete job
for an empty list of items. probably means that the ItemDeleteJob
class should assert on such cases so we find other errors like this.

Patch by Ahti, thanks!


MERGE: 4.7

Andras Mantia committed changes in [kdepim] /:

Make mailfilter's dummykernel less dummy, and also provide a Settings interface to it.
This properly fixes 286762 (the previous fix just postponed the crash to a later stage).
Remove MailCommon::Kernel::settingsAreRegistered, as it doesn't make sense to have it (SettingsIf is mandatory).

Allen Winter committed changes in [kdepim] /:

Don't attempt to access the kernel settings if the kernel hasn't
been registered yet.


FIXED-IN: 4.7.4
MERGE: 4.7

Office

Jarosław Staniek committed changes in [calligra] /:

Restore "Reports: fix bug: no indication of selected toolbox action for report element"
+fix for Plan





This reverts commit b424b352f682db0b31e68f1d8b35cf55f5c13693.

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 Revision 940fced...
Thorsten Zachmann committed changes in [calligra] libs/flake/KoConnectionShape.cpp:

Fix saving of connections that are part of a group

This fixes bug 288108 - Connection in group not saved correctly

Sebastian Sauer committed changes in [calligra] /tables:

Fix chart vs plotArea background-color.

Before we did not proper evaluate the plotArea background-color and
opacity in 2000/2003/2007 charts and where ending in lot of cases
with the wrong plotArea background-color for that reason.

Gopalakrishna Bhat committed changes in [calligra/lists-gopalK] /:

Fix bug 282091:Words not saves heading numbering. Now we save text:outline-style at office:styles.

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 (+ 4 more) Revision 8914970...
Gopalakrishna Bhat committed changes in [calligra] /textlayout:

Fix for bug 287747: Underline is not displayed for numbered list labels. Now these are displayed

Thorsten Zachmann committed changes in [calligra] libs/flake/KoUnavailShape.cpp:

Create valid odf on saving of Unavail shape

This fixes bug 288138 - Unavail shape creates invalid xml on saving

Allan Anderson committed changes in [kmymoney] kmymoney/plugins/csvimport/investprocessing.cpp:

If a CSV file contains lines with differing numbers of columns (or the
last column is empty), and if one of the selected columns happens to
be the last column in a line with fewer than the maximum number,
an array goes out of bounds.

Use the actual line's column list count instead of maximum column count.

Sebastian Sauer committed changes in [calligra] plugins/chartshape/PlotArea.cpp:

Fix axis orientation was swapped when the diagram was vertical.

This can be verified by loading the ODP referenced at bug 287895
into Stage. There at slide number 9 we displayed only one big
bar rather then all the bars with the expected at the y-axis
defined chart:maximum. That was so cause we completly ignored
any vertical=true definitions for the axes cause Axis::loadOdf
was called way before we called PlotArea::setVertical.

C. Boemann committed changes in [calligra] libs/textlayout/KoTextLayoutArea.cpp:

Fix dropcaps that contain a softbreak. In the same step we also fix
dropcaps that begin with spaces

Adam Pigg committed changes in [calligra] /libs/ui/reports:

Updated plan to properly uncheck report items once inserted

Sebastian Sauer committed changes in [calligra] tables/Sheet.cpp:

Fix embedded objects are not saved if they are after the last row with content.
This is the second and last part to address and finally close bug 287997. The both commits are independent of each other (means they both fix independent bugs that where visible with the ODS document attached to the bugreport).

Sven Langkamp committed changes in [calligra] /:

fix locking of tools, when the method returned before the the end reached the layer stays locked
using KisSystemLocker everywhere now to prevent such cases

Matus Uzak committed changes in [calligra] /:

libmsooxml: libmsooxml: fine tuned text:line-break processing

Introducing a new <text:span> element having text-properties
defined by endParaRPr in selected situations in order to prevent
usage of the styleForModification function:

* Do not save text-properties into paragraph styles also in case
of empty paragraphs. Save an empty <text:span> having
text-properties defined by endParaRPr. Looking for the default
font size defined by the application in case the endParaRPr
formatted span is removed and new text is inserted.

* Insert an empty <text:span> having text-properties defined by
endParaRPr in case the <text:line-break> is the last element of
the paragraph in order to apply the correct fo:font-size to the
new line. Replaced the original fo:font-size from
text-properties of <text:line-break> by TEXT_FONTSIZE_MIN to
not affect the line-height calculation of both the new line and
the current line. This disables of course application of the
default font-size defined by the application in case of
<text:line-break>.

* Removing the fo:text-transform attribute is also removed from
text-properties of text:line-break because it disables its
functionality during layout (did not check the layout part).

* Also removing the style:text-underline-style attribute. If
text styles of the text chunk and text:line-break match,
underline is applied until end of the line during layout.

Multimedia

Rick Chen committed changes in [amarok] /:

Disable cover fetching action for tracks not in the collection


FIXED-IN: 2.5

Rick Chen committed changes in [amarok] /services:

Fix service albums cover not being set after successful download

Weak qobject pointers cannot be promoted to qsharedpointer. So in
coverDownloadComplete() the downloaded cover is never used. Just use
ServiceAlbumWithCoverPtr which is available.

Sam Lade committed changes in [amarok] /:

Fix last.fm scrobble composer fallback to artist

Now correctly scrobbles with the artist name if the composer field is empty and
the scrobble composer option is enabled, rather than silently failing.

Patch by Ilya Kasnacheev.


FIXED-IN:2.5
REVIEW:103279

Sven Krohlas committed changes in [amarok] /:

Correctly parse disambiguation pages for songs in the English Wikipedia

The regexp looks wrong, I gues it didn't work at all before...


FIXED-IN:2.5.0

Sven Krohlas committed changes in [amarok] src/context/engines/wikipedia/WikipediaEngine.cpp:

Wikipedia applet: regexp for German disambiguation pages

Why was that missing? That's one of our largest user bases...

Also, many, many more languages need to be added there.

Networking Tools

Lamarque Souza committed changes in [networkmanagement/nm09] /:

Do not use null backend pointer in warning message.

Lamarque Souza committed changes in [networkmanagement] applet/nmpopup.cpp:

Change mobile broadband (wwan) checkbox into a tristate checkbox. I do
not know why NM sends a wwan disabled signal right after we send an
enable wwan command, the checkbox bounces for while until NM finally
decides to enable or not the mobile broadband state.

The third state is "partically checked", represented by a dashed tick
mark.

Yoann Laissus committed changes in [rekonq] src/application.cpp:

GCI Task : Don't open useless tabs or windows for actions in the new tab page
Patch by Hüseyin Zengin. Thanks !
REVIEW: 103257

Lamarque Souza committed changes in [networkmanagement] /:

Caches deviceUni and activatableType for RemoteActivatable class to
prevent dbus calls from stalling the entire desktop.

Diffs: 1, 2, 3, 4 Revision f633cc1...
Matthias Fuchs committed changes in /trunk/KDE/kdenetwork/kget/mainwindow.cpp:

Makes dropping urls on KGet work in more cases.
Thanks go to Dawit for the patch.

Eike Hein committed changes in [konversation] src/main.cpp:

Clean up line breaks in app description.

Lamarque Souza committed changes in [networkmanagement] /:

Change networkmanagement_configshell to inform error using dialogs
instead of console messages.

Joris Guisson committed changes in [libktorrent] /:

Fix crash in PeerManager

Other

Arno Rehn committed changes in [smokegen] type_compiler.cpp:

fix a crash when generating the akonadi bindings

Cristian Oneț committed changes in [kmymoney] /:

Since the register has a mechanism to make the selected item visible remove the external
viewport vertical scroll position setting mechanism from kgloballedgerview since the two
mechanisms conflicted with each other causing this bug. This was the code is also simplified
and we can assume that scrollTo does not change the viewport's position if the item is visible.

Features

Development Tools

Ralf Habacker committed changes in /trunk/KDE/kdesdk/umbrello/umbrello/codeimport:

added support for msvc2010 preprocessor (windows only)

On the first instance creation available preprocessors are detected and saved.
If neither cpp and cl could not be found in the pathes provided by
KStandardDirs::findExe(), then the default install location of msvc2010 is
searched for.

Aleix Pol Gonzalez committed changes in [kdevelop] /:

Make it possible to define some environment variables to be run
on cmake executions.

These environment variables are integrated both in the CMake parsing
and the actual cmake executions inside KDevelop.

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 (+ 5 more) Revision 1de7caa...
Sven Brauch committed changes in [kdev-python] /:

Improved support for comprehensions

List comprehension is fully supported now, except for
highlighting the iterated variable, which requires DUChain changes.

Dict comprehensions are supported, but without content type
tracking. The target is only assigned a plain "dict" type.

Diffs: 1, 2, 3, 4, 5 Revision d16c01f...
BogDan Vatra committed changes in [android-qt/keyboard-refatoring] /:

Finishing software keyboard refactoring.

Close the following issues:

Graphics

Marcel Wiesweg committed changes in [digikam] /dimg:

Provide methods to remove an alpha channel from an image. Remove alpha channel before saving to JPEG.

- hasTransparentPixels checks if, given there is an alpha channel,
some pixels actually is transparent (alpha < 1.0)
- bitBlendImageOnColor allows to compose an image over a constant color
This is equivalent to create a DImg, fill it with a color,
blend the first image over the new one, and replace the first image with the result.
- removeAlphaChannel blends the image on a color, usually, white, and removes the alpha channel
- in updateMetadata:
ensure that the previous embedded preview is always removed
only add a new check that a preview is only added if the original image is significantly
larger than the intended preview size
To be discussed and currently unchanged: handling of transparent pixels

KDE Base

Lamarque Souza committed changes in [kde-workspace/ksmserver/qml-shutdowndlg] /:

Show a QML dialog on KDE session exiting. There are two themes:

1. default.qml, this is a basic replacement for the current dialog. It
is not fully functional yet. I need to add the boot options (for grub2)
and work more on the GUI since the QML components I am using lack some
features used in the current dialog (QPushButton popup menu, icons on
button's right side, growing background effect on QPushButtons, label
text aligned to the right instead of to left). The dialog/shutdowndialog
borders also do not work properly, dialog's resizing still needs work
and the screen's graying out effect does not work. I guess the QML
dialog is stopping the graying out effect somehow.

2. contour.qml, this is meant to be used with Plasma Active. Contour.qml
uses larger buttons for touchscreen friendliness. For now it includes
only the lock screen, sleep and shutdown butons.

contour.qml is the default in this patch, that will change when this
branch is merged to kde-workspace master branch.

Diffs: 1, 2, 3 Revision 009ceeb...
Ivan Čukić committed changes in [kdelibs/ivan/kparts-activities] /:

Initial support for resource events in read-only kparts

This makes Okular play nicely with activities and
Share-Like-Connect

Diffs: 1, 2, 3, 4, 5, 6, 7, 8 Revision 0b017f3...
Michał Dutkiewicz committed changes in /trunk/playground/base/plasma/applets/miniplayer:

Update to 1.1:
- added toggle play list button to default controls set;
- added track length infromation to play list (sortable);
- added option to disable D-Bus interace;
- improved dialog appearance;
- more friendly install script;
- removed compile time dependency on gettext;
- some other fixes.

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 (+ 8 more) Revision 1266928
Aaron J. Seigo committed changes in [kde-workspace] plasma/desktop/shell/plasmaapp.cpp:

re-enable auto-migration when screens change

hopefully this is fixed by e2108ed to libplasma; if not we can restore the #if 0 for 4.8.0

Peter Penz committed changes in [kde-baseapps] /src:

Reimplement name-filtering

The filtering of items has not been implemented yet in
the KFileItemModel of the new view-engine. The patch brings back
this functionality again, but some minor issues are open:
- When filtering trees expanded directories should only get
hidden if no child is visible
- Regular expressions are not supported yet (they have not been
supported in Dolphin 1.x but it is now quite simple to implement).
- When filtering previews and removing the filter it might be
possible that the preview is not shown (is most probably an
an already existing bug in KFileItemModelRolesUpdater).


FIXED-IN: 4.8.0

Thomas Lübking committed changes in [kde-workspace] kwin/effects/taskbarthumbnail/taskbarthumbnail.cpp:

apply effect transformations on taskbar thumbnail

Aaron J. Seigo committed changes in [kde-workspace] plasma/generic/applets/icon/icon.cpp:

if it is a binary, then make a desktop file for it

patch by Will Stephenson

Martin Gräßlin committed changes in [kde-workspace/kwin/tabbox-window-strip] /tabbox:

Embedded mode for TabBox

DBus method to embedd the TabBox into another window. It follows
the geometry changes and keeps a defined offset to the borders of
the parent window.

Required for Plasma Active's window strip.

Diffs: 1, 2, 3, 4, 5, 6 Revision aaf5728...
Martin Gräßlin committed changes in [plasma-mobile] /:

Better support for KWin's TabBox feature

TabBox is now completely controlled from Plasma side. It gets
activated when the dragging starts and connects to the item selected
signal. When an item got selected the panel is closed. As soon as
the panel closed either accept or reject of TabBox is invoked to
close it and (if an item was selected) to activate the window.

This means the switching is slightly delayed.

Diffs: 1, 2, 3, 4 Revision 91d5576...

KDE-PIM

Laurent Montel committed changes in [kdepim] /:

First step to allow to export/import filter (don't save as a collection

id but real path and we will restore it)

Diffs: 1, 2, 3, 4, 5, 6 Revision 3700e9f...
Shaheed Haque committed changes in /trunk/playground/pim/akonadi/exchange:

Start of attempt to support writing to the server:

1. Implement a set of class-based wrappers which can be declared
on the stack, and thus automatically deal with resource deallocation
of all the myriad error paths. These could be used throughout the
code to the same end.

2. Implement comprehensive error reporting logic in support of the
above.

3. Flesh out the calendar change path, but leave it disabled. The logic
in mapiconnector2.cpp exposes a bug in libmapi where SetProps asserts in
cast_mapi_SPropValue() because it omits support for PT_MV_CLSID.

Diffs: 1, 2, 3, 4 Revision 1267143

Office

Stephane Mankowski committed changes in [skrooge/Feature] /:

feature: Dashboard > Shares & Indexes renamed to "Quotes" and provide a possibility to select all available Units (Objects, Currencies etc.)

Diffs: 1, 2, 3, 4, 5, 6 Revision 702ee97...
Stephane Mankowski committed changes in [skrooge/Feature] /:

feature: Export of non local files (example: http://myserver/document.qif)

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9 Revision 55ac5ab...
Sebastian Sauer committed changes in [calligra] /:

Implement support for filled radar charts.

This commit does;
1. implement support for filled radar charts in KDChart. The same patch was applied to KDChart master so the change is not lost when we sync later with a newer KDChart version.
2. implement support for reading, writing and editing of filled radar charts in the charting plugin.
3. implement support for reading filled radar chart from Microsoft 2000/2003/2007 documents and producing proper ODF for filled radar charts out of it which is then passed on to the charting plugin.

Jarosław Staniek committed changes in [calligra] /:

Kexi: improved opening of .kexis files and creating new app instances

Details:
*Do not display connection dialog when opening .kexis file that define database name
*Welcome view: when clicked on a database open it in a new Kexi instance if another database is already opened in the current instance

DIGEST: Kexi: improved opening of .kexis files and creating new app instances

Diffs: 1, 2, 3, 4, 5, 6, 7 Revision 5eed0f2...

Multimedia

Sven Krohlas committed changes in [amarok] src/services/amazon/AmazonParser.cpp:

Show cover for all tracks from the Amazon store in the playlist

The API is soon going to support this, this makes use of that change.

Colin Guthrie committed changes in /branches/KDE/4.7/kdemultimedia/kmix/backends/mixer_pulse.cpp:

kmix: Add support for capture stream volumes introduced in PulseAudio 1.0

Sven Krohlas committed changes in [amarok] src/services/amazon/AmazonStore.cpp:

Amazon store: More intelligent searching for album of a track

If it's a compilation: don't add the artist name to the search string.

Networking Tools

Andrea Diamantini committed changes in [rekonq] /:

Shortcut + slot to open downloads page (in a new focused tab)

Google code-in 2011 task by Omer Faruk Oruc.
Thanks!!!

REVIEW: 103207
REVIEWED-BY: furkan, adjam

Andrea Diamantini committed changes in [rekonq/DownloadManagement] /:

Download Management ++

- let users choose a "default" download dir
- Moved download code to download manager
- follow dfaure's suggestions about overwrite handling
(kdelibs commit 88e3b372fb9c539)
- download label notification (for 3 seconds): this can be further
improved with an icon...

Martin Klapetek committed changes in [telepathy-text-ui] /:

Add spell dictionary combobox

Reviewed-by: David Edmundson
REVIEW: 103296

Utilities

Rolf Eike Beer committed changes in [kgpg] /:

Implemented group inplace editing.

[REB:did some coding style edits, but changed no functionality]

Other

Laszlo Papp committed changes in [qtopenal] src/qalsndaudiodecoder.cpp:

Implement the seek, tell and read callbacks inside the snd audio decoder

Optimization

KDE Base

Marco Martin committed changes in [kde-runtime] /:

more transparent minimized task, they stick out more



M +- -- desktoptheme/air/widgets/tasks.svgz

Thomas Lübking committed changes in [kde-workspace] kwin/effects/magiclamp/magiclamp.cpp:
Sebastian Trueg committed changes in [nepomuk-core/symlinkHandling] /:

Improved symlink handling on the indexer level.

The indexer will now index directly symlinked files like any other file.
Indirectly symlinked files will create an additional kext:altUrl
property on the target file.
For this to actually work kext:altUrl will have to be treated as a 2nd
special case besides nie:url in the DMS.

Diffs: 1, 2, 3 Revision c88aa10...
Sebastian Kügler committed changes in [plasma-mobile] applications/webbrowser/package/contents/ui/content/FlickableWebView.qml:

don't smooth image while scaling the webpage during zoom

I'm not sure if this is *really* needed. Rendering is disabled, so the
webpage won't do layout updates which makes for the biggest performance
gain already. The image still seems scaled smoothly though, so we
disable that as well and just scale the frames during zoom animations
coarsely.

There's no visible performance gain on my laptop, but let's try anyway
if it squeezes out a few more FPS on devices.

Aaron J. Seigo committed changes in [kdelibs/KDE/4.7] plasma/windoweffects.cpp:

don't allocate 1k chunks of memory for a few longs here and there

Aaron J. Seigo committed changes in [kde-workspace] /desktop/applets/kickoff/core:

thread the disk free info fetching so on systems where it is slow it doesn't hang the whole UI

dolphin may wish to do similarly?

KDE-PIM

Christian Mollekopf committed changes in [kdepim-runtime] /nepomukfeeder:

Avoid using to much memory and overloading nepomuk

The ItemQueue stores now id's only instead of full items, and only fetches a couple of items at a time.
This ensures that not too much memory is used. To ease the load on nepomuk if a storejob times out,
we insert a timeout.

Laurent Montel committed changes in [kdepim] /:

Optimization. Don't create foldercollection when not necessary. Show error message when we delete folder (before we didn't see it because folderCollection was deleted before)

Clean up

Diffs: 1, 2, 3, 4, 5 Revision a3bf760...
David Faure committed changes in [kdepim] messagelist/core/modelinvariantrowmapper.cpp:

Major speedup when deleting many messages.

Turns out that QHash::begin() is very slow, it's a linear search through
the hash's buckets, to find the starting point.

Office

Dag Andersen committed changes in [calligra] /workpackage:

Improve document handling

Display new document name property
Add new document(s) on merge
Add remove document command

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9 Revision 369b160...
Sebastian Sauer committed changes in [calligra/plugins-uservariables-sebsauer] /:

Introduce and use optimized version of KoXml::namedItemNS to lookup for KoXmlTextContentPrelude elements.
The saving isn't that huge. If I take out the optimization then the lookup takes ~3ms with the OO.org 1.1 specs and ~1ms with the patch.
This addresses a review-comment from https://git.reviewboard.kde.org/r/103276/

Diffs: 1, 2, 3 Revision 0b5253e...
Sebastian Sauer committed changes in [calligra] tables/DependencyManager.cpp:

Optimize DependencyManager by using iteraters to save some lookups.

On loading the www.worldmapper.org%2Fdata%2Funderlying%2FWorldmapper_U3_hdr%304_tables1-22.ods document the DependencyManager::updateAllDependencies took before ~20.2 seconds and takes now ~18.8 seconds what equals a speedup of ~1.5 seconds.

Jarosław Staniek committed changes in [calligra] /:

Kexi's Main Window main area: show close buttons on tabs; remove borders

Details:
*set close buttons on view tabs always visible
*remove main area borders for cleaner GUI
*remove MainWindow.HoverCloseButtonForTabs setting

DIGEST: Kexi's Main Window main area: show close buttons on tabs; remove borders

Diffs: 1, 2, 3, 4 Revision 842e90f...
Sebastian Sauer committed changes in [calligra] /:

Optimize the inverse iterator by making it by a factor of 100 faster.

The inverse iterator logic is used in statistical functions like betainv. Before we parsed and applied a formula >=1000 times for each single iterator call. That is slow^as^hell. This is fixed now by passing in a function-pointer for the formula function that should be used during iteration what obsoletes the expensive^as^hell formula execution done before.

The effect is that documents that made heavy using of formulas using the inverse iterator are now a few hundred factors faster loaded and displayed. This is most notable with the www.inrets.fr%2Ffileadmin%2Fur%2Fma%2FFichiers_mistral%2FPage_Brenac%2FContr-Bf-Aft-Accid-Study2.ods from our collection of test-documents. Before loading that file never finished (cstester aborted with signal 9) while now we have it loaded and displayed in <6 seconds.

Diffs: 1, 2, 3 Revision 816d498...
Sebastian Sauer committed changes in [calligra] /:

Optimize Formula::scan and make it close to double as fast.

Regular expression using QRegExp are indeed evil. This improves the time spend in Formula::scan during loading the www.worldmapper.org%2Fdata%2Funderlying%2FWorldmapper_U3_hdr%304_tables1-22.ods from ~4.6 seconds to ~2.8 seconds what saves us around ~2 seconds.
Tested with cstester and some Q_ASSERT's against all our ods, xls and xlsx documents to make sure the just introduced Util::isCellReference behaves exactly like the regular expression did before.

Diffs: 1, 2, 3, 4 Revision 8bb8684...
Sebastian Sauer committed changes in [calligra] /:

Optimize saving to ODF.

This combines the commits ecac0d03d90c and the idea from commit 43b0a39eb91 together to;
1) keep the to-cell anchored shape handling intact and working as expected.
2) improve the saving-speed by some tons of factors by using the RowRepeatStorage to compress identical rows together.

All tested test-documents are still working as expected before vs after the patch. Feedback welcome :)

Multimedia

Rick Chen committed changes in [amarok] /:

lastfm: load friend/neighbour avatars on demand

Otherwise all your friends' and neighbours' avatars are downloaded every
time when starting amarok (cached and whatnot). May not be much of an
issue depending on how popular you are.

Diffs: 1, 2, 3 Revision bd113ce...
Matěj Laitl committed changes in [amarok] /:

Fix infinite loop in context view eating CPU until main window is shown

We used to use _scene_ sceneRect in
VerticalToolbarContainment::updateGeometry() to update applets and
geometry, but that leaded to infinite loop (across mainloop) -
m_applets->setGeometry(), refresh() would enlarge _scene_ sceneRect by
a few pixels which would trigger updateGeometry() and so on...

We now use _view_ sceneRect to update geometry. Rick W. Chen discovered
some glitches in applet collapsing when this patch is applied, but he
has a patch for that in the works and this is more serious problem, so
pushing this.

REVIEW: 103271

FIXED-IN: 2.5

Networking Tools

Andrea Diamantini committed changes in [rekonq] src/rekonqui.rc:

rekonq menu clean up (last hopefully)

This change comes from comparison with firefox/chrome menus

Andrea Diamantini committed changes in [rekonq] src/rekonq.kcfg:

Faster clean startup (with just two side previews to load)

Other

Office

Matus Uzak committed changes in [calligra] /:

libmsooxml: text properties inheritance kung fu

* Don't add style:use-window-font-color="true" into
text-properties of a style of a text chunk when fo:color is not
provided. This only affects text inside tables at the moment,
becasue the default text colors (color scheme defined by
slideLayout) are used for any other text. This was overriding
the fo:color defined in the cell style.

* Remove all text-properties from paragraph style in case the
paragraph is not empty. The endParaRPr element specifies the
text run properties that are to be used if another run is
inserted after the last run specified. It does not affect
formatting of text chunks in the paragraph. That's a paragraph
specific piece of information that we are not going to save
into ODF at the moment.

* Store <text:line-break> inside of a <text:span> element.
Process the rPr (Text Run Properties) of the br (Text Line
Break) element, but only use it to format the
<text:line-break> (It should apply to a newly inserted run of
text). That's another piece of information that we are not
going to save into ODF at the moment.

> I inferred the following a inheritance hierarchy:
1. font-size defined in the rPr element inside br
2. minimal font-size in the paragraph if any font size is provided
3. font-size defined by endParaRPr if present
(other text-properties are not important at the moment)

REASON: At the moment the current text style is applied until
end of the line (like underline) and then the line-break is
applied. Also when a line-break is located at the end of
paragraph (empty line is inserted) the application default
font-size is used because there are no text-properties in the
pragraph style.

Diffs: 1, 2, 3, 4, 5 Revision f084b60...