<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>392504</bug_id>
          
          <creation_ts>2018-03-29 21:16:48 +0000</creation_ts>
          <short_desc>The channels docker/layer properties dialog does not show the locked channels properly</short_desc>
          <delta_ts>2018-04-03 11:46:28 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>krita</product>
          <component>General</component>
          <version>git master (please specify the git hash!)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Halla Rempt">halla</reporter>
          <assigned_to name="Krita Bugs">krita-bugs-null</assigned_to>
          
          
          <cf_commitlink>https://commits.kde.org/krita/c8103fa9f264405e95c481bfeda102561da216c7</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1741738</commentid>
    <comment_count>0</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2018-03-29 21:16:48 +0000</bug_when>
    <thetext>For this file, the channels docker/layer properties dialog does not reflect the actual channel lock settings: https://drive.google.com/file/d/1xQtDR2jzmddVoL_T20AyHVJYYAvMUmdL/view?usp=sharing

I am not sure whether the original author saved the file with certain channels locked, but the file contains this:

   &lt;layer y=&quot;-307&quot; nodetype=&quot;paintlayer&quot; name=&quot;classic rake.jpg&quot; colorspacename=&quot;RGBA&quot; channelflags=&quot;&quot; channellockflags=&quot;1011&quot; selected=&quot;true&quot; onionskin=&quot;0&quot; uuid=&quot;{22b1a997-901c-4b97-9368-f1c8e215126c}&quot; locked=&quot;0&quot; collapsed=&quot;0&quot; compositeop=&quot;normal&quot; colorlabel=&quot;0&quot; filename=&quot;layer3&quot; opacity=&quot;255&quot; intimeline=&quot;0&quot; visible=&quot;1&quot; x=&quot;-1827&quot;/&gt;

And it is certainly loaded like this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1741841</commentid>
    <comment_count>1</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2018-03-30 07:37:18 +0000</bug_when>
    <thetext>Git commit 7a27f04cbe093ff1880a261f38504c2dfa0800d1 by Boudewijn Rempt.
Committed on 30/03/2018 at 07:33.
Pushed by rempt into branch &apos;master&apos;.

Do not save and load channel lock flags to .kra

Until the channel docker was made to work on the projection in

    commit 365d0eb775cbff3bf6afda3f9b703741dbd68710
    Author: Boudewijn Rempt &lt;boud@valdyas.org&gt;
    Date:   Mon Sep 30 09:02:41 2013 +0200

    Start making the channel docker work on the projection, not the layer

We could lock channels and painting on them would be disabled. Except
for some remnants in libqml, that is no longer possible. However, we
would still load and save the channel lock flags. Maybe in the future
we should add the channel lock flags to the layer properties dialog
instead and restore this code.

I&apos;m still not sure how the file in the bug report was created, unless
it originated in Krita Gemini.

M  +0    -82   libs/libqml/plugins/kritasketchplugin/models/LayerModel.cpp
M  +0    -16   libs/libqml/plugins/kritasketchplugin/models/LayerModel.h
M  +0    -49   libs/libqml/qml/panels/EditLayerPage.qml
M  +0    -3    plugins/impex/libkra/kis_kra_loader.cpp
M  +0    -5    plugins/impex/libkra/kis_kra_savexml_visitor.cpp
M  +0    -1    plugins/impex/libkra/kis_kra_tags.h

https://commits.kde.org/krita/7a27f04cbe093ff1880a261f38504c2dfa0800d1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1742844</commentid>
    <comment_count>2</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2018-04-03 11:46:28 +0000</bug_when>
    <thetext>Git commit c8103fa9f264405e95c481bfeda102561da216c7 by Boudewijn Rempt.
Committed on 03/04/2018 at 11:20.
Pushed by rempt into branch &apos;krita/4.0&apos;.

Do not save and load channel lock flags to .kra

Until the channel docker was made to work on the projection in

    commit 365d0eb775cbff3bf6afda3f9b703741dbd68710
    Author: Boudewijn Rempt &lt;boud@valdyas.org&gt;
    Date:   Mon Sep 30 09:02:41 2013 +0200

    Start making the channel docker work on the projection, not the layer

We could lock channels and painting on them would be disabled. Except
for some remnants in libqml, that is no longer possible. However, we
would still load and save the channel lock flags. Maybe in the future
we should add the channel lock flags to the layer properties dialog
instead and restore this code.

I&apos;m still not sure how the file in the bug report was created, unless
it originated in Krita Gemini.
(cherry picked from commit 7a27f04cbe093ff1880a261f38504c2dfa0800d1)

M  +0    -82   libs/libqml/plugins/kritasketchplugin/models/LayerModel.cpp
M  +0    -16   libs/libqml/plugins/kritasketchplugin/models/LayerModel.h
M  +0    -49   libs/libqml/qml/panels/EditLayerPage.qml
M  +0    -3    plugins/impex/libkra/kis_kra_loader.cpp
M  +0    -5    plugins/impex/libkra/kis_kra_savexml_visitor.cpp
M  +0    -1    plugins/impex/libkra/kis_kra_tags.h

https://commits.kde.org/krita/c8103fa9f264405e95c481bfeda102561da216c7</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>