Issue 214
18th December 2011 by KDE Commit-Digest TeamContributors
Jaka Kranjc
Marta Rybczynska
Paulo Dias
Danny Allen
This Week...
Statistics
Commits | 1720 by 183 developers |
Open Bugs | 22106 |
Open Wishes | 17262 |
Bugs Opened | 512 in the last 7 days |
Bugs Closed | 876 in the last 7 days |
Commit Summary
Module | Commits |
/trunk/l10n-kde4 |
240
|
/trunk/KDE |
82
|
/trunk/www |
44
|
/branches/stable |
33
|
/trunk/l10n-support |
31
|
/branches/work |
19
|
/plasma/desktop |
14
|
/dolphin/src |
13
|
/akonadi/calendar |
13
|
/trunk/playground |
12
|
Files | Developer | Commits |
234
|
Gilles Caulier |
79
|
210
|
Ralf Habacker |
71
|
147
|
Laurent Montel |
50
|
111
|
Mohamad Reza Mirdamadi |
37
|
105
|
Aaron J. Seigo |
35
|
75
|
Marco Martin |
34
|
93
|
Lamarque Souza |
32
|
75
|
Albert Astals Cid |
25
|
72
|
Yuri Chornoivan |
24
|
72
|
Dennis Nienhüser |
24
|
Internationalization (i18n) Status
Bug Killers
Person | Bugs Closed |
Gilles Caulier |
157
|
Peter Penz |
125
|
Jekyll Wu |
80
|
Dawit Alemayehu |
49
|
Andrea Diamantini |
36
|
sebsauer kdab com |
35
|
Aaron J. Seigo |
35
|
Thijs Heus |
30
|
Christoph Feck |
29
|
Myriam Schweingruber |
29
|
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 102 selections this week
Bug Fixes
Development Tools
Fix OptionAST precondition.
Graphics
Use a queued connection instead of direct call
It seems some shells (like konqueror and kile) are too smart and play with the focus and the focus ends up in the wrong place because they "overwrite" our setFocus call, now we workaoround that by using the queuedconnection
FIXED-IN: 4.8.0
KDE Base
When embedding a new part, do not try to hide the current parts' widget.
REVIEW: 103401
FIXED-IN: 4.8.0
fix gravitation after compositor "polluted" it by updating the decoration _before_ placement
REVIEW: 103391
manager the lifespan of the icon dialog to match that of the activity icon
Update the roles if items have been changed
The code "// TODO..." in slotItemsChanged() obviously was not
sufficient ;-)
FIXED-IN: 4.8.0
The slider should react whenever the value changes, and not just on move.
Thanks Amos Kariuki for suggesting the fix
(cherry picked from commit 57307a33f3cd6872c9f136e47b13336560f2ab5c)
Changed maxInstances property to 20 and maxInstancesPerHost to 5.
Do not enable the "Link View" action initially.
FIXED-IN: 4.8.0
Fix several sort-issues
- Treeview: When sorting descending assure that the parent item is still
ordered before the child items and not afterwards.
- Treeview: When sorting by other roles than names expansionsLevelCompare()
had been buggy and resulted in ordering child items below wrong parent
items.
- General: When sorting by another role than names and the role of
two items had been equal a case sensitive sorting of the names had
been done. This has been fixed by using the default name sorting
as fallback.
FIXED-IN: 4.8.0
use font metrics to decide if the text is enabled
using the kayout size hint causes a loop because the hint will be smaller when the text is off
KLocale: Fix readNumber() and readMoney() for lenient group parsing
Restore the old pre-4.7 behaviour of accepting as valid any numbers
that do not contain any group separators but strictly enforce group
rules when the number contains 1 or more group separators.
Distro's will really want to backport this fix to all versions of 4.7
as previously number entry for all KDE apps would have been seriously
broken.
I'm amazed I wasn't beaten up for this earlier!
FIXED-IN: 4.7.5
ignore mouse presses
Start an anchored selection when clicking a selection marker
FIXED-IN:4.8.0
delay showing popups until the next processing of events
allows things like hidden panels to show before calculating where to show the popup
* Use KIO::copyAs and KIO::moveAs to rename/move files so that the
user is properly prompted when an existing name is used.
* Changed assert calls to Q_ASSERT calls.
FIXED-IN: 4.8.0
Open links in external browser instead of loading then in about app's
WebView.
KDE-PIM
Fix Bug 289100 - Pegasus Mail Import crashes regularly in kmailcvt
FIXED-IN: 4.8
First part of the fix for bug 285118:
- when creating the initial list of choices for the combobox, exclude
those timezones we are already have chosen from user prefs
- when adding a view timezone, remove that from the combobox choices
so the user can't add it again
- when removing a view timezone, add that back to the combobox choices
in messageItemBefore(), in the case that there is no current item
and starting from the deepest item, make sure that deepest item
has a parent else there will be a crash.
FIXED-IN: 4.8.0
Start to fix Bug 282721 - no filtering on sent mails
FIXED-IN: 4.8
Office
Disable the per-input-device switch of krita tools
I already (in eab5746f9f94b48f67ec20cc6d28e9221c53fef1) disabled the
switching of tools when switching input devices. This patch makes sure
we don't create tools for new input devices either, which fixes a problem
with the tool action shortcuts being reset to default when switching between
mouse and tablet.
guard against resizing shapes in the direction where the shape size is zero
Disable MyMoneyFile signals while we import a GnuCash file.
This is needed to avoid updating views while the import is not
finished and the data in the MyMoneyFile is not ready to be used.
Do not create rdf objects if the xml:id doesn't start with rdfid-
Following the deprecation of draw:id and text:id, elements that used to
just save draw:id and text:id now also save these id's as xml:id.
Calligra's rdf support thinks that every xml:id is a reference to some
rdf metadata. This breaks because there _is_ no rdf data associated.
Until I've found a proper solution to shared xml:id's between various
parts of calligra, check whether the xml:id starts with rdfid- and only
then load the metadata.
Yes, this probably means we can no longer load documents with rdf saved
by other applications like abiword. I'm working on a "proper" solution.
Fix importing and exporting of multi-line note fields using Alexandria
Tellico was not using the YAML multi-line scalar correctly.
FIXED-IN: 2.3.5
don't crash in the RTL painting strategy if the canvas is qgraphicsview based
Fix writing one image tag twice to an odf document.
Fix mem-leak.
Don't leak the just created new auto-KoCharacterStyle which is only temporary used to generate a proper auto-style while saving the QTextCharFormat.
The mem-leak was introduced in e66889d566e.
add workaround to prevent color chooser to jumb to state that was used before picking a color
Adam please check
Fix crash if either the Sheet or the SheetView got destroyed without letting the View know about it.
This commit makes sure we proper remove the Sheet and it's associated SheetView from the sheetViews-QHash the View holds if the Sheet is deleted. Also it makes sure that if the SheetView is deleted then we proper set it to NULL so we do not deal with dangling pointers what would lead to a crash latest if the View itself is destroyed.
Fix styles are changed during saving.
This is especially fatal if the used KoCharacterStyle is the default KoCharacterStyle in which case we modify the character-style properties in a rather random way just because we try to save the content.
The bug was introduced with commit e66889d566e. Verified that bug 260115 the commit fixed is not reopened and things still work as expected.
KisSelection: delete default bounds object if it hasn't a parent
It's conceivable that a selection is createde but never used, in which
case the default bounds object isn't passed to to a pixel selection,
in which case it doesn't have a parent, and isn't deleted.
So if the defaultbounds object doesn't have a parent, explicitly delete it.
Fix tables crashed when removing a row
Fix italic and underline default values.
The spec says for them "If omitted, the default value is true." what means it's italic/bold if val is not defined.
One step closer to fix bug 286158. But there still seems to be a problem with Tables not proper rendering/diplay bold fonts.
Multimedia
Fix bogus logic when opening KWallet in Last.fm configuration
The logic is still a bit convoluted, but at a last-minute fix this
suffices and should introduce no problems.
FIXED-IN: 2.5
iPod handler: do not crash when master playlist is null
This is a late fix for bug 288936. I've added i18n( "Unknown" ) during
string freeze, but this exact string is already used elsewhere.
Sorry for such a last-minute commit, but this was an easy crashfix.
FIXED-IN: 2.5
media device collection: get rid of CustomReadCapability - was unused
Since Bart ported UMS, this class was not instantiated anywhere, so
remove the cruft.
Networking Tools
Do NOT search for (probably) non extant favicons
also use KIO::copy instead of KIO::file_copy to delegate file
management to KIO
This should hopefully fix||workaround HTML code shown bug. Anyway,
I cannot understand why KIO merges the two different jobs data together
Fixed memory leak craeted by actions
new actions were created every time the context menu was called. Now context menu is created
immediatly and kept only once in memory
show history entries also when no favorite search engine is set
Bug investigation and courtesy patch by Gabo Menna.
Many thanks for :D
Utilities
put compression and encryption of a folder into a KJob
This fixes an empty KPassivePopup showing up in the top left of the screen,
which is a remnant of KDE 3.5 days. It also improves the feedback to the user
on what's happening a lot.
And while at it this also fixes a couple of memleaks and other stuff.
FIXED-IN:4.8.0
Games
FIXED IN: KDE SC 4.8 Ensure that markers are displayed after a Load action.
FIXED-IN: KDE SC 4.8
REVIEW: 6839
Fixes selecting the wrong stone if clicking on a shadow of a stone.
Other
use stricter refing in all places of 'identity' keeping.
it is interesting that it would be much harder
to accidentaly make same mistake if G API was C++ style and not C.
i also moved cleanup of successful sessions in time from system shutdown
to session completion as polkit docs say it should be
Features
Development Tools
Add support for properties appending in set_property.
added import feature
If enabled in the new code import settings page for each imported c++ file
an artifact will be created in the component view.
The rational of this feature - when it is completed - is to generate files
completly from the parsed content including file header comments and the
exact collection of which classes belongs to a file.
To achieve this, a new class FileAST has been created, which holds all
file related informations not fitting into other declarations.
The implementation creates artifacts with the raw file name because there
are problems in Model_Utils::findUMLObject searching full pathes.
To complete this feature the following is to do:
- UMLArtifact has to be extended to hold uml objects collected for this file.
- ArtifactWidget has to be extended to let users editable the lists provided
by UMLArtifact.
- The parser should add the collected uml objects to the related UMLArtifact
- The artifacts in the component view should be structured into folders using
abolute pathes or based on the import root (switchable in the code import
settings page)
a try to implement iteration over the list view items using keyboard
Replace Projects QComboBox by Filtered QListView in KDE Provider
It should help developers find projects faster
REVIEW: 103383
GUI: QComboBox now replaced by a filtered QListView
Educational
Create a dialog to modify one or more books.
Class ModifyDialog allows the user to edit data of one or more than
one book. Special "Previous" and "Next" button are provided,
to navigate through the entries.
There is also a checkbox which allows the user to edit multiple kind
of data at the same time, for all selected books.
Signed-off-by: Riccardo Bellini
Graphics
update internal libraw to 0.14.4 release :
- Fixes to Panasonic/Leica file parser to prevent crash on broken jpegs.
- Fixes to include order in src/libraw_datastream.cpp to better compile with KDEWIN
- Floating-point DNGs are rejected on early processing stage.
- Support for new cameras: Canon S100, Fuji X10, Panasonic GX1, Samsung NX200, Sony NEX-7.
in icon-view mode, zoom button named "Fit to Window" can be used now to fill all icon rows with thumb contents automaticaly. The right thumbs size is computed accordingly with space available...
CC
new wrapper to extract ISO speed info from more tags, especially makernotes
delete dialog is now able to display thumbnails of items to delete
KDE Base
Port to QTemporaryDir
add support for graphicssystem selection
REVIEW: 103430
allow controlling the resize a bit more
this allows e.g. popupapplet to put constraints on where the edges of the window can resize to
and react differently when the user is actively resizing the dialog.
Add keyboard navigation support to qml-shutdowndlg. Use the TAB key to
nagivate between the buttons and the ENTER key to trigger the action. I
was not able to make the first button be the "zero TAB key". You have to
press TAB once to make the first button the first TAB key and a second
time to go to the second button.
Highlight window: support minimized windows
also
- replace QHash::operator[]
- replace double by float (-> ARM, and we don't need that precision)
- weaken some value checks ("== 1.0f")
- fix last paint of hiding windows (w->addRepaintFull() trap)
REVIEW: 103404
KDE-PIM
Refactor and rewrite KGoogleAuth and AuthDialog.
KGoogleAuth is now KGoogle::Auth (in auth.h and auth.cpp). It
supports multiple Google accounts and acts as a frontend to a
secure storage (KWallet in this case). It still provides methods
for authenticating new accounts or refreshing access tokens to
an existing one, the API is much more simple and straightforward though.
Make reply prefix case-insensitive
Introduce KGoogle::Account class and common.h
This class represents a single Google account, holds authentication tokens
and list of scopes the tokens are valid for.
Introduce common.h as header with common values and objects. For now it
contains KGoogle::Error enumarator and few basic exceptions
The "KGoogle" in name was dropped since all classes are in KGoogle namespace
so there is no need to have KGoogle::KGoogleSomeClass.
Inline TNEF support is Back!
The separate MS-TNEF viewer tool 'ktnef' is gone so
in format():
- don't ever return with AsIcon since we no longer have ktnef
- don't make winmail.dat a hyperlink since we no longer have ktnef
- deal with empty TNEF attachments -- keep a placeholder in the
message to show that we do have an attachment, albeit an empty one.
this is ugly, but probably best not to make the user think that
the attachment is lost somehow.
in subtype():
- support subtype vnd.ms-tnef
Add command line parameters support to kmail-mobile.
REVIEW: 8985
Office
Extend undo/redo functionality.
Extend the ChangeImageCommand class to be able to undo/redo changes
done to the cropping area and the color mode.
Updated to use DIscogs API v2
Some deprecated endpoints are still used, in order to have XML
results. Full usage of API v2 will require using JSON.
allow adding zotero contacts to akonadi
Implement rich-text support for table-cells for Excel 2000/2003 documents.
Add support for draw:text-areas in msooxml autoshapes
Tested with cstester. No regression found, only positive changes.
Highlight the online statement balance if different from current file balance
REVIEW: 103264
if file already exists, save resource with a new name
Support ODF axis font-colors.
1) support for sorted bibliography
2) using default entry templates instead of creating one in BibliographyGenerator
Multimedia
Collection browser: show absolute free space on hover, thicker bar
For collections that support capacity (iPod, UMS, MTP), change text on
capacity bar from "80% used" to "1,2 GiB free". Rationale:
* relative emptiness is already indicated using bar
* absolute free space is the most relevant information when
transferring tracks
* this is consistent with Dolphin
Additionally, the capacity bar is made thicker and the font is non-bold
so that the font is readable when rendered on it. (this is done by
using KCapacityBar's minimumSizeHint() - however it is bounded between
MIN and MAX values)
This may be a little controversial change of the look, here are some
screenshots, speak up if you don't like it:
* before: http://troll.ws/i/VanuSm.png
* after: http://troll.ws/i/RTSrAq.png
DIGEST: feature: more useful capacity bar when hovering collection
Amazon store artwork, part 1: logo and emblem
Thanks to "Fabi" (whatever your real name is).
To make them show up I had to delete some caches:
/var/tmp/kdecache-$USER/icon-cache.kcache
and
/var/tmp/kdecache-$USER/Amarok-pixmaps.kcache
So if the icons don't show up try that.
License:
"since i dont know much about licences.. choose whatever you want :)"
So we make that GPLv2.1 or later.
M +- -- src/images/emblem-amazon.png
M +- -- src/images/icons/hi16-action-view-services-amazon-amarok.png
M +- -- src/images/icons/hi22-action-view-services-amazon-amarok.png
M +- -- src/images/icons/hi32-action-view-services-amazon-amarok.png
M +- -- src/images/icons/hi48-action-view-services-amazon-amarok.png
INital try to fix missing controls. Needs verification from affected users.
CC
Networking Tools
Enable setting presence message for auto-away
Reviewed-by: Dario Freddi
REVIEW: 103417
rekonq 0.8.56
- sync! (just with ftp sites for now...)
- bookmarkprovider --> bookmarkmanager
- docs --> doc
- etc...
DIGEST: sync feature merged in rekonq master
Add filtering for filetransfer-capability and make filtering more generic
Add the possibility to filter for contacts with filetransfer capability
as it is needed for send-file.
Filtering for name now respects showOfflineUsers.
REVIEW: 103409
Support multiple file transfers
Add a filter bar to quickly find contacts
REVIEW: 103390
Games
FIXED-IN: KDE SC 4.8
REVIEW: 6837
After shuffle the stones because no moves left, you will now get a dialog about "No moves left" if the game isn't solvable.
Optimization
Development Tools
Improve split_quoted_on_whitespace.
Apparently Perl has included (since version 5.0, no less) a module that
handles almost the exact case split_quoted_on_whitespace exists for.
The impetus for this is that the '${MODULE}' syntax for dest-dir is
apparently broken due to problems with the mixture of braces and quotes
in split_quoted_on_whitespace.
This change does not break the test suite (but does add a relevant test)
and does seem to fix the issue with strings with braces.
KDE-PIM
Improve performance by:
1. Eliminate use of strings for ids in mapiconnector2.
2. Use the shortest/densest string format when storing ids in Akonadi.
Also, move the implmentation of MapiItem into the .cpp file.
Office
use official icons for visibility
Don't initialize ChartShape with default values
to then just overwrite them in loading. This speeds up loading by
30%-50%. The worldmapper file does load with that change 5 seconds
faster 25 seconds comared to 30 seconds.
Improve performance while scrolling around the sheet attached to bug 281910.
Intel's VTunes shows 3 hotsports. This commit fixes two of them. The theird and yet not fixed case is CellStorage::isPartMerged which does an expensive RTree-lookup. Probably we should cache that info?
Optimize formula parsing by a factor of 2-4
This optimizes formula parsing by a factor of 2-4. The file
www.worldmapper.org%2Fdata%2Funderlying%2FWorldmapper_U3_hdr%304_tables1-22.ods
loads now in 25 instead of 30 seconds.
The patch tries to avoid creating QString objects when not necessary and reuse
QString object when possible. The reduces the number of allocations quite a lot.
Also implicit casts of 'x' and "foo" are avoided where ever possible as these
are also quite expensive.
e.g. some of the easy optimizations are
if (QString::operator[0] == '$')
is better written as
if (QString::operator[0] == QChar('$', 0)
as this can be optimized already be the compiler and thus reduces the stuff needed
otherwise.
Also initialize class members in initialization list of the constructor where possible
as this avoids copying the data a second time which is also expensive if it is e.g. a
QString.
Additionally to the speed up also the formula parsing now can recognice a escaped " in
a formula and parsing of error codes has been updated to follow ODF 1.2. This fixes the
failing unit tests in TestFormula.
Added unit test for decodeFormula.
This code also contains some debug code to check that the new results is actually the
same as the old results. Due to the fixes in the formula parsing that might no longer
be true for some of the formulas. This check will be removed in the next commit so that
it is easy to get it back if more testing is needed. When running against a set of ods,
xls and xlsx it showed no difference before fixing the formula parsing.
Networking Tools
minor startup speed improvement (QL1C && QL1S)
Other
Development Tools
In cmake, Don't reuse declarations if they are not in a cmake script.
Graphics
usability issue : use whole timeline cursor to make date selection, and not only bottom sqare.
CC
KDE Base
Use a QApplication and a KComponentData instead of a KApplication.
This removes the KApplication DBus interface which allows us to
control how the services are shut down, ie. deleting the service itself
before shutting down the QCoreApplication.
get rid of 'title' attribute in DocBook generated HTML
fixes annoying tooltips in KDE documentation
REVIEW:103405
Make string more understandable for users
Users confused Kontact events with holiday events.
Got the permission from i18n
No need to link to kdecore anymore
Multimedia
Amazon store artwork, part 3: cart icons
The shpping cart icons are there. :-) Wohoooooooo!
-> We are ready for release.
Popup dropper SVGs are not yet included, I have to check out how that
beast works. But that can be done for 2.6.
Thanks again Fabi for the great work.
Btw: I had to re-run cmake to make the build scripts fetch up those
new icons. So might you.
A +- -- src/images/icons/hi16-action-amarok_cart_add.png
A +- -- src/images/icons/hi16-action-amarok_cart_remove.png
A +- -- src/images/icons/hi16-action-amarok_cart_view.png
A +- -- src/images/icons/hi22-action-amarok_cart_add.png
A +- -- src/images/icons/hi22-action-amarok_cart_remove.png
A +- -- src/images/icons/hi22-action-amarok_cart_view.png
A +- -- src/images/icons/hi32-action-amarok_cart_add.png
A +- -- src/images/icons/hi32-action-amarok_cart_remove.png
A +- -- src/images/icons/hi32-action-amarok_cart_view.png
A +- -- src/images/icons/hi48-action-amarok_cart_add.png
A +- -- src/images/icons/hi48-action-amarok_cart_remove.png
A +- -- src/images/icons/hi48-action-amarok_cart_view.png
A +- -- src/images/icons/svg/action-cart_add.svgz
A +- -- src/images/icons/svg/action-cart_remove.svgz
Refactor the FreeBSD SCSI/CAM interface.
>From the review request:
Main idea of the change is to improve the case where SCSI sense data
is not automatically provided and has to be explicitly requested.
Current code essentially duplicates main transport code for this task.
The proposed code recursively calls into the transport code with
MMC_REQUEST_SENSE command. This also fixes a problem with the existing
code where it re-uses a CCB of the original command for sense fetching
but doesn't ensure that all the previously used bytes are reset to
proper values.
This can result in a malformed MMC_REQUEST_SENSE CCB which can confuse
certain hardware (e.g. it hangs Optiarc DVD RW AD-7191S 1.02). Also the
style of the code is cleaned up.
Because of the code re-use the code is now more compact. Additionally
some historic and useless code was dropped - the code for setting errno.
errno value is not used by the calling code and this is an artifact of
the FreeBSD-specific code having been borrowed from a different project
(as attested by Heiner Eichmann).
Patch by Andriy Gapon, thanks a lot!
REVIEW: 103293
(cherry picked from commit dfe605a68406bd8830d3ab6c6e228f2c387a4122)
Networking Tools
Ask for secrets only once per connection.
FIXED-IN: 0.9.0_rc4
Replaced all occurrences of FeedbackWidget with KMessageWidget.
Also turned two Informative messages into Error messages to make it consistent.
Signed-off-by: Dario Freddi
REVIEW:103398