Only expose ApplyPageBehaviors

PrismLibrary · Jan 18, 2020 · 73b4fd1 · 73b4fd1
1 parent 09f946c
commit 73b4fd1
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 42 deletions.
diff --git a/Source/Xamarin/Prism.Forms/Behaviors/IPageBehaviorFactory.cs b/Source/Xamarin/Prism.Forms/Behaviors/IPageBehaviorFactory.cs
@@ -13,38 +13,5 @@ public interface IPageBehaviorFactory
         /// <param name="page">The page to apply the behaviors</param>
         /// <remarks>The PageLifeCycleAwareBehavior is applied to all pages</remarks>
         void ApplyPageBehaviors(Page page);
-
-        /// <summary>
-        /// Applies behaviors to a CarouselPage.
-        /// </summary>
-        /// <param name="page">The CarouselPage to apply the behaviors</param>
-        /// <remarks>The CarouselPageActiveAwareBehavior is applied by default</remarks>
-        void ApplyCarouselPageBehaviors(CarouselPage page);
-
-        /// <summary>
-        /// Applies behaviors to a ContentPage.
-        /// </summary>
-        /// <param name="page">The ContentPage to apply the behaviors</param>
-        void ApplyContentPageBehaviors(ContentPage page);
-
-        /// <summary>
-        /// Applies behaviors to a MasterDetailPage.
-        /// </summary>
-        /// <param name="page">The MasterDetailPage to apply the behaviors</param>
-        void ApplyMasterDetailPageBehaviors(MasterDetailPage page);
-
-        /// <summary>
-        /// Applies behaviors to a NavigationPage.
-        /// </summary>
-        /// <param name="page">The NavigationPage to apply the behaviors</param>
-        /// <remarks>The NavigationPageSystemGoBackBehavior and NavigationPageActiveAwareBehavior are applied by default</remarks>
-        void ApplyNavigationPageBehaviors(NavigationPage page);
-
-        /// <summary>
-        /// Applies behaviors to a TabbedPage.
-        /// </summary>
-        /// <param name="page">The TabbedPage to apply the behaviors</param>
-        /// <remarks>The TabbedPageActiveAwareBehavior is added by default</remarks>
-        void ApplyTabbedPageBehaviors(TabbedPage page);
     }
 }
diff --git a/Source/Xamarin/Prism.Forms/Behaviors/PageBehaviorFactory.cs b/Source/Xamarin/Prism.Forms/Behaviors/PageBehaviorFactory.cs
@@ -12,7 +12,7 @@ public class PageBehaviorFactory : IPageBehaviorFactory
         /// </summary>
         /// <param name="page">The CarouselPage to apply the behaviors</param>
         /// <remarks>The CarouselPageActiveAwareBehavior is applied by default</remarks>
-        public virtual void ApplyCarouselPageBehaviors(CarouselPage page)
+        protected virtual void ApplyCarouselPageBehaviors(CarouselPage page)
         {
             page.Behaviors.Add(new CarouselPageActiveAwareBehavior());
         }
@@ -21,26 +21,26 @@ public virtual void ApplyCarouselPageBehaviors(CarouselPage page)
         /// Applies behaviors to a ContentPage.
         /// </summary>
         /// <param name="page">The ContentPage to apply the behaviors</param>
-        public virtual void ApplyContentPageBehaviors(ContentPage page)
+        protected virtual void ApplyContentPageBehaviors(ContentPage page)
         {
-            
+
         }
 
         /// <summary>
         /// Applies behaviors to a MasterDetailPage.
         /// </summary>
         /// <param name="page">The MasterDetailPage to apply the behaviors</param>
-        public virtual void ApplyMasterDetailPageBehaviors(MasterDetailPage page)
+        protected virtual void ApplyMasterDetailPageBehaviors(MasterDetailPage page)
         {
-            
+
         }
 
         /// <summary>
         /// Applies behaviors to a NavigationPage.
         /// </summary>
         /// <param name="page">The NavigationPage to apply the behaviors</param>
         /// <remarks>The NavigationPageSystemGoBackBehavior and NavigationPageActiveAwareBehavior are applied by default</remarks>
-        public virtual void ApplyNavigationPageBehaviors(NavigationPage page)
+        protected virtual void ApplyNavigationPageBehaviors(NavigationPage page)
         {
             page.Behaviors.Add(new NavigationPageSystemGoBackBehavior());
             page.Behaviors.Add(new NavigationPageActiveAwareBehavior());
@@ -50,8 +50,15 @@ public virtual void ApplyNavigationPageBehaviors(NavigationPage page)
         /// Applies behaviors to a page based on the page type.
         /// </summary>
         /// <param name="page">The page to apply the behaviors</param>
-        /// <remarks>The PageLifeCycleAwareBehavior is applied to all pages</remarks>
-        public virtual void ApplyPageBehaviors(Page page)
+        /// <remarks>
+        /// There is no need to call base.ApplyPageBehaviors when overriding.
+        /// All Prism behaviors have already been applied.
+        /// </remarks>
+        protected virtual void ApplyPageBehaviors(Page page)
+        {
+        }
+
+        void IPageBehaviorFactory.ApplyPageBehaviors(Page page)
         {
             switch (page)
             {
@@ -73,14 +80,15 @@ public virtual void ApplyPageBehaviors(Page page)
             }
 
             page.Behaviors.Add(new PageLifeCycleAwareBehavior());
+            ApplyPageBehaviors(page);
         }
 
         /// <summary>
         /// Applies behaviors to a TabbedPage.
         /// </summary>
         /// <param name="page">The TabbedPage to apply the behaviors</param>
         /// <remarks>The TabbedPageActiveAwareBehavior is added by default</remarks>
-        public virtual void ApplyTabbedPageBehaviors(TabbedPage page)
+        protected virtual void ApplyTabbedPageBehaviors(TabbedPage page)
         {
             page.Behaviors.Add(new TabbedPageActiveAwareBehavior());
         }