Bug #2398

Strange message in debuglog if you run youtube plugin

Added by Leonid Protasov over 6 years ago. Updated over 6 years ago.

Status:FixedStart date:12/02/2014
Priority:NormalDue date:
Assignee:Andreas Smas% Done:

0%

Category:General
Target version:4.8
Found in version:4.7.527 Platform:Linux

Description

prop            [ERROR]: Refusing to clean prop clone.focused<0x7f2a3c03f280> because a decendant have canonical subs

Associated revisions

Revision fe42bae9
Added by Andreas Smas over 6 years ago

prop: Improve the 'Refusing to clean prop ...' error message

refs #2398

Change included in version 4.7.561

Revision a750ff35
Added by Andreas Smas over 6 years ago

glw: Add (currently disabled) code to do proper unlink when using links for assignment

This is currently not needed but might be so in the future, so keep the code around

refs #2398

Change included in version 4.7.562

History

#1 Updated by Andreas Smas over 6 years ago

  • Status changed from New to Accepted

I don't think this is much to worry about. It's probably somehow related to the youtube .view files and maybe something in them are a bit broken.

I will try to look into it a bit

#2 Updated by Andreas Smas over 6 years ago

Here is how to fix it:

diff --git a/youtube/views/array2.view b/youtube/views/array2.view
index d6c6a64..a303432 100644
--- a/youtube/views/array2.view
+++ b/youtube/views/array2.view
@@ -47,7 +47,7 @@ widget(container_y, {
                                widget(array, {
                                        $view.sizer = int(getWidth() / 100);

-                                       &clone.focused = focusedChild();
+                                       &clone.focused ?= select(isFocused(), focusedChild(), void);
                                        .id = "array";
                                        .border = $view.sizer * 3;
                                        .Xspacing = $view.sizer;
@@ -113,4 +113,4 @@ widget(container_y, {
                LIST_Y_SLIDER("array");
                widget(dummy, { .width = 2; });
        });
-});
\ No newline at end of file
+});
diff --git a/youtube/views/header.view b/youtube/views/header.view
index 1b38506..69a84a8 100644
--- a/youtube/views/header.view
+++ b/youtube/views/header.view
@@ -9,7 +9,7 @@ widget(container_y, {
                widget(container_x, {
                widget(list_x, {
                        .spacing = 5;
-                       &clone.focused = focusedChild();
+                       &clone.focused ?= select(isFocused(), focusedChild(), void);
                        $view.change = changed($clone.focused, 1);

                        cloner($page.model.actions, container_z, {
@@ -82,4 +82,4 @@ widget(container_y, {
                });
                });
        });*/
-});
\ No newline at end of file
+});

This also makes the help text to the upper right correctly respond when moving between the header and the array. (Which kinda is the bug in this case)

#3 Updated by Andreas Smas over 6 years ago

  • Status changed from Accepted to Fixed

Also available in: Atom PDF