Make WeakEventHandlerManager public

2 posts, 0 answers
  1. Wetzorke
    Wetzorke avatar
    22 posts
    Member since:
    Aug 2012

    Posted 17 Mar Link to this post

    Hi,

     

    we are currently having problems with premature execution of CanExecute (System.Windows.Input.ICommand.CanExecute(object)) on our custom asynchronous ICommand implementation. Unfortunately, we cannot base our asynchronous ICommand on your DelegateCommand (Telerik.Windows.Controls.DelegateCommand). Our research has shown that this problem (and other problems like memory leaks) can be solved by using a WeakEventHandlerManager implementation. Frameworks like Prism are implementing such a WeakEventHandlerManager.

    In general, we would like to keep our dependencies in our projects as low as possible, hence avoid referencing Prism or other libraries implementing a WeakEventHandlerManager. We have found in several forum posts (see below), that UI for WPF is implementing the internal static class WeakEventHandlerManager (Telerik.Windows.Controls.WeakEventHandlerManager), but is inaccessible to us due to its protection level.

    http://www.telerik.com/forums/custom-appointment-validation#31FR5-6S9UGnhsD0Fcbzyw

    http://www.telerik.com/forums/radmediaplayer-system-nullreferenceexception-object-reference-not-set-to-an-instance-of-an-object#3aeiMSrBEEGTu9wz1x4Wcw

    http://www.telerik.com/forums/getfirstpartofoccurrenceswithnoenddates-leads-to-argumentoutofrangeexception-when-adding-no-end-date-recurring-appt-at-daystarttime-in-dayview#ERl6dJHd20yu02FAW4VkXw

    http://www.telerik.com/forums/random-com-exception#RAVYUzxVokKq-46Gm1Q9qA

     

     

    I am asking you to change the visibility of your WeakEventHandlerManager to public, so we can benefit from it without introducing an additional dependency in our projects just for the sake of this one helper class.

    Please see the associated Feature Request: https://feedback.telerik.com/Project/143/Feedback/Details/213908-make-weakeventhandlermanager-public

     

    Best Regards

  2. Ivan Ivanov
    Admin
    Ivan Ivanov avatar
    1201 posts

    Posted 22 Mar Link to this post

    Hi,

    Indeed this looks like a valuable improvement for the Telerik WPF suite. I am changing the status of the feedback item to "Approved". You can follow it in order to track which of the future releases will contain the change. I am adding 1000 Telerik points to your account for submitting this feature request.

    Regards,
    Ivan Ivanov
    Telerik by Progress
    Want to extend the target reach of your WPF applications, leveraging iOS, Android, and UWP? Try UI for Xamarin, a suite of polished and feature-rich components for the Xamarin framework, which allow you to write beautiful native mobile apps using a single shared C# codebase.
Back to Top