┌   ┐
54
└   ┘

summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSebastian Trueg <[email protected]>2007-05-14 09:21:58 +0000
committerSebastian Trueg <[email protected]>2007-05-14 09:21:58 +0000
commit2abfe6b6926a16eeabc1f580a837159972c979ed (patch)
tree07944760277bfe748486a9b2fbcf3c1dc2cccbd3
parent19744a5ee16e8b0e51a49bfc96414141c7361d2e (diff)
Try not to set properties unless changed.
svn path=/trunk/KDE/kdebase/apps/; revision=664542
-rw-r--r--src/metadatawidget.cpp16
1 files changed, 11 insertions, 5 deletions
diff --git a/src/metadatawidget.cpp b/src/metadatawidget.cpp
index 3ef7bec7d..52b33093d 100644
--- a/src/metadatawidget.cpp
+++ b/src/metadatawidget.cpp
@@ -122,8 +122,6 @@ void MetaDataWidget::setFile(const KUrl& url)
// FIXME: replace with KMetaData::File once we have it again
d->fileUrl = url;
d->file = Nepomuk::KMetaData::Resource(url.url(), s_nfoFile);
-// d->file.setLocation(url.url());
- d->file.setProperty( s_nfoFileUrl, url.url() );
d->ratingWidget->setRating(d->file.rating());
d->tagWidget->setTaggedResource(d->file);
d->loadComment(d->file.description());
@@ -135,7 +133,7 @@ void MetaDataWidget::setFiles(const KUrl::List urls)
{
#ifdef HAVE_KMETADATA
// FIXME: support multiple files
- setFile(urls.first());
+ setFile( urls.first() );
#endif
}
@@ -143,7 +141,11 @@ void MetaDataWidget::setFiles(const KUrl::List urls)
void MetaDataWidget::slotCommentChanged()
{
#ifdef HAVE_KMETADATA
- d->file.setDescription(d->editComment->toPlainText());
+ if ( d->editComment->toPlainText() != d->file.description() ) {
+// d->file.setLocation(url.url());
+ d->file.setProperty( s_nfoFileUrl, d->fileUrl.url() );
+ d->file.setDescription(d->editComment->toPlainText());
+ }
#endif
}
@@ -151,7 +153,11 @@ void MetaDataWidget::slotCommentChanged()
void MetaDataWidget::slotRatingChanged(int r)
{
#ifdef HAVE_KMETADATA
- d->file.setRating(r);
+ if ( r != d->file.rating() ) {
+ // d->file.setLocation(url.url());
+ d->file.setProperty( s_nfoFileUrl, d->fileUrl.url() );
+ d->file.setRating(r);
+ }
#endif
}