extragear/graphics/digikam/showfoto

[F.J.Cruz]

SVN commit 493895 by fjcruz:

Bug fixed: if "Default Path" for icc files in Color Management setup dialog is empty and user try to enable color management, this one is disabled and an error message is showed .

CCMAIL:[hidden email]

 M  +15 -1     setupicc.cpp  


--- trunk/extragear/graphics/digikam/showfoto/setupicc.cpp #493894:493895
@@ -232,6 +232,10 @@
     KConfig* config = kapp->config();
 
     config->setGroup("Color Management");
+    if (!m_enableColorManagement->isChecked())
+    {
+        return;
+    }
     config->writeEntry("EnableCM", m_enableColorManagement->isChecked());
     if (m_defaultApplyICC->isChecked())
     {
@@ -239,11 +243,21 @@
     }
     else
     {
-        config->writeEntry("BehaviourICC", false);
+        config->writeEntry("BehaviourICC", "false");
     }
 //     config->writeEntry("ApplyICC", m_defaultApplyICC->isChecked());
 //     config->writeEntry("AskICC", m_defaultAskICC->isChecked());
+    if (m_defaultPath->url().isEmpty())
+    {
+        QString message = QString(i18n("<p>You must set a default path to color profiles files.</p>"));
+        message.append(i18n("<p>This settings will not be written.</p>"));
+        KMessageBox::error(this, message );
+        config->writeEntry("EnableCM", false);
+        config->sync();
+        return;
+    }
     config->writeEntry("DefaultPath", m_defaultPath->url());
+//     config->writeEntry("DefaultPath", m_defaultPath->url());
     config->writeEntry("WorkSpaceProfile", m_workProfiles->currentItem());
     config->writeEntry("MonitorProfile", m_monitorProfiles->currentItem());
     config->writeEntry("InProfile", m_inProfiles->currentItem());
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Op dinsdag 3 januari 2006 16:39, schreef Francisco J. Cruz:
> SVN commit 493895 by fjcruz:
>
> Bug fixed: if "Default Path" for icc files in Color Management setup dialog
> is empty and user try to enable color management, this one is disabled and
> an error message is showed .

Hi Paco,

> -        config->writeEntry("BehaviourICC", false);
> +        config->writeEntry("BehaviourICC", "false");

Did you mean to commit this?

Toma

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel