💥 Diferencia entre renderizar y preparar página

pagetop-admin/src/lib.rs

@@ -30,7 +30,7 @@ impl ModuleTrait for Admin {
     }
 
     fn actions(&self) -> Vec<Action> {
-        vec![action!(ActionBeforeRenderPage => before_render_page)]
+        vec![action!(ActionBeforePreparePage => before_prepare_page)]
     }
 
     fn configure_service(&self, cfg: &mut service::web::ServiceConfig) {
@@ -40,6 +40,6 @@ impl ModuleTrait for Admin {
     }
 }
 
-fn before_render_page(page: &mut Page) {
+fn before_prepare_page(page: &mut Page) {
     page.alter_body_classes(ClassesOp::Add, "test-admin");
 }

pagetop-aliner/src/lib.rs

@@ -21,7 +21,7 @@ impl ModuleTrait for Aliner {
 }
 
 impl ThemeTrait for Aliner {
-    fn before_render_page(&self, page: &mut Page) {
+    fn before_prepare_page(&self, page: &mut Page) {
         page.alter_favicon(Some(Favicon::new().with_icon("/theme/favicon.ico")))
             .alter_context(ContextOp::AddStyleSheet(
                 StyleSheet::located("/aliner/css/styles.css").with_weight(-99),

pagetop-bootsier/src/lib.rs

@@ -42,7 +42,7 @@ impl ThemeTrait for Bootsier {
         ]
     }
 
-    fn before_render_page(&self, page: &mut Page) {
+    fn before_prepare_page(&self, page: &mut Page) {
         page.alter_favicon(Some(Favicon::new().with_icon("/theme/favicon.ico")))
             .alter_context(ContextOp::AddStyleSheet(
                 StyleSheet::located("/bootsier/css/bootstrap.min.css")
@@ -57,7 +57,7 @@ impl ThemeTrait for Bootsier {
         pagetop_jquery::JQuery::add_jquery(page.context());
     }
 
-    fn render_page_body(&self, page: &mut Page) -> Markup {
+    fn prepare_page_body(&self, page: &mut Page) -> Markup {
         match page.template() {
             "admin" => html! {
                 body class=[page.body_classes().get()] {
@@ -66,22 +66,22 @@ impl ThemeTrait for Bootsier {
                         "side-menu",
                         "content"
                     ] {
-                        @if let Some(content) = page.render_region(region) {
+                        @if let Some(content) = page.prepare_region(region) {
                             #(region) { (content) }
                         }
                     }
                 }
             },
             _ => {
-                let header = page.render_region("header");
+                let header = page.prepare_region("header");
-                let nav_branding = page.render_region("nav_branding");
+                let nav_branding = page.prepare_region("nav_branding");
-                let nav_main = page.render_region("nav_main");
+                let nav_main = page.prepare_region("nav_main");
-                let nav_additional = page.render_region("nav_additional");
+                let nav_additional = page.prepare_region("nav_additional");
-                let breadcrumb = page.render_region("breadcrumb");
+                let breadcrumb = page.prepare_region("breadcrumb");
-                let content = page.render_region("content");
+                let content = page.prepare_region("content");
-                let sidebar_first = page.render_region("sidebar_first");
+                let sidebar_first = page.prepare_region("sidebar_first");
-                let sidebar_second = page.render_region("sidebar_second");
+                let sidebar_second = page.prepare_region("sidebar_second");
-                let footer = page.render_region("footer");
+                let footer = page.prepare_region("footer");
                 html! {
                     body class=[page.body_classes().get()] {
                         @if header.is_some() {

pagetop-bulmix/src/lib.rs

@@ -30,7 +30,7 @@ impl ModuleTrait for Bulmix {
 }
 
 impl ThemeTrait for Bulmix {
-    fn before_render_page(&self, page: &mut Page) {
+    fn before_prepare_page(&self, page: &mut Page) {
         page.alter_favicon(Some(Favicon::new().with_icon("/theme/favicon.ico")))
             .alter_context(ContextOp::AddStyleSheet(
                 StyleSheet::located("/bulmix/css/bulma.min.css")

pagetop-node/src/lib.rs

@@ -27,7 +27,7 @@ impl ModuleTrait for Node {
     }
 
     fn actions(&self) -> Vec<Action> {
-        vec![action!(ActionBeforeRenderPage => before_render_page, -1)]
+        vec![action!(ActionBeforePreparePage => before_prepare_page, -1)]
     }
 
     fn migrations(&self) -> Vec<MigrationItem> {
@@ -44,6 +44,6 @@ async fn node(request: service::HttpRequest) -> ResultPage<Markup, FatalError> {
     Page::new(request).with_title(L10n::n("Nodo")).render()
 }
 
-fn before_render_page(page: &mut Page) {
+fn before_prepare_page(page: &mut Page) {
     page.alter_body_classes(ClassesOp::Add, "test-node");
 }

pagetop/src/base/theme/basic.rs

@@ -21,7 +21,7 @@ impl ModuleTrait for Basic {
 }
 
 impl ThemeTrait for Basic {
-    fn before_render_page(&self, page: &mut Page) {
+    fn before_prepare_page(&self, page: &mut Page) {
         page.alter_favicon(Some(Favicon::new().with_icon("/theme/favicon.ico")));
     }
 }

pagetop/src/core/theme/definition.rs

@@ -21,13 +21,9 @@ pub trait ThemeTrait: ModuleTrait + Send + Sync {
     }
 
     #[allow(unused_variables)]
-    fn before_render_page(&self, page: &mut Page) {
+    fn before_prepare_page(&self, page: &mut Page) {}
-        if page.favicon().is_none() {
-            page.alter_favicon(Some(Favicon::new().with_icon("/theme/favicon.ico")));
-        }
-    }
 
-    fn render_page_head(&self, page: &mut Page) -> Markup {
+    fn prepare_page_head(&self, page: &mut Page) -> Markup {
         let title = page.title();
         let description = page.description();
         let viewport = "width=device-width, initial-scale=1, shrink-to-fit=no";
@@ -64,11 +60,11 @@ pub trait ThemeTrait: ModuleTrait + Send + Sync {
         }
     }
 
-    fn render_page_body(&self, page: &mut Page) -> Markup {
+    fn prepare_page_body(&self, page: &mut Page) -> Markup {
         html! {
             body class=[page.body_classes().get()] {
                 @for (region, _) in self.regions().iter() {
-                    @if let Some(content) = page.render_region(region) {
+                    @if let Some(content) = page.prepare_region(region) {
                         #(region) { (content) }
                     }
                 }
@@ -76,6 +72,12 @@ pub trait ThemeTrait: ModuleTrait + Send + Sync {
         }
     }
 
+    fn after_prepare_page(&self, page: &mut Page) {
+        if page.favicon().is_none() {
+            page.alter_favicon(Some(Favicon::new().with_icon("/theme/favicon.ico")));
+        }
+    }
+
     #[rustfmt::skip]
     #[allow(unused_variables)]
     fn before_render_component(

pagetop/src/response/page.rs

@@ -1,7 +1,10 @@
 pub use actix_web::Result as ResultPage;
 
-mod action;
+mod before_prepare_page;
-pub use action::{ActionBeforeRenderPage, ACTION_BEFORE_RENDER_PAGE};
+pub use before_prepare_page::{ActionBeforePreparePage, ACTION_BEFORE_PREPARE_PAGE};
+
+mod after_prepare_page;
+pub use after_prepare_page::{ActionAfterPreparePage, ACTION_AFTER_PREPARE_PAGE};
 
 mod definition;
 pub use definition::Page;

pagetop/src/response/page/after_prepare_page.rs

@@ -2,25 +2,25 @@ use crate::core::action::{ActionTrait, AnyAction};
 use crate::response::page::Page;
 use crate::{use_handle, Handle};
 
-use_handle!(ACTION_BEFORE_RENDER_PAGE);
+use_handle!(ACTION_AFTER_PREPARE_PAGE);
 
 type Action = fn(&mut Page);
 
-pub struct ActionBeforeRenderPage {
+pub struct ActionAfterPreparePage {
     action: Option<Action>,
     weight: isize,
 }
 
-impl ActionTrait for ActionBeforeRenderPage {
+impl ActionTrait for ActionAfterPreparePage {
     fn new() -> Self {
-        ActionBeforeRenderPage {
+        ActionAfterPreparePage {
             action: None,
             weight: 0,
         }
     }
 
     fn handle(&self) -> Handle {
-        ACTION_BEFORE_RENDER_PAGE
+        ACTION_AFTER_PREPARE_PAGE
     }
 
     fn weight(&self) -> isize {
@@ -32,7 +32,7 @@ impl ActionTrait for ActionBeforeRenderPage {
     }
 }
 
-impl ActionBeforeRenderPage {
+impl ActionAfterPreparePage {
     pub fn with_action(mut self, action: Action) -> Self {
         self.action = Some(action);
         self

pagetop/src/response/page/before_prepare_page.rs

@@ -0,0 +1,51 @@
+use crate::core::action::{ActionTrait, AnyAction};
+use crate::response::page::Page;
+use crate::{use_handle, Handle};
+
+use_handle!(ACTION_BEFORE_PREPARE_PAGE);
+
+type Action = fn(&mut Page);
+
+pub struct ActionBeforePreparePage {
+    action: Option<Action>,
+    weight: isize,
+}
+
+impl ActionTrait for ActionBeforePreparePage {
+    fn new() -> Self {
+        ActionBeforePreparePage {
+            action: None,
+            weight: 0,
+        }
+    }
+
+    fn handle(&self) -> Handle {
+        ACTION_BEFORE_PREPARE_PAGE
+    }
+
+    fn weight(&self) -> isize {
+        self.weight
+    }
+
+    fn as_ref_any(&self) -> &dyn AnyAction {
+        self
+    }
+}
+
+impl ActionBeforePreparePage {
+    pub fn with_action(mut self, action: Action) -> Self {
+        self.action = Some(action);
+        self
+    }
+
+    pub fn with_weight(mut self, weight: isize) -> Self {
+        self.weight = weight;
+        self
+    }
+
+    pub fn run(&self, page: &mut Page) {
+        if let Some(action) = self.action {
+            action(page)
+        }
+    }
+}

pagetop/src/response/page/definition.rs

@@ -5,8 +5,9 @@ use crate::core::theme::ComponentsRegions;
 use crate::html::{html, Classes, ClassesOp, Favicon, Markup, DOCTYPE};
 use crate::locale::{langid_for, LanguageIdentifier};
 use crate::response::fatal_error::FatalError;
-use crate::response::page::action::{ActionBeforeRenderPage, ACTION_BEFORE_RENDER_PAGE};
 use crate::response::page::ResultPage;
+use crate::response::page::{ActionAfterPreparePage, ACTION_AFTER_PREPARE_PAGE};
+use crate::response::page::{ActionBeforePreparePage, ACTION_BEFORE_PREPARE_PAGE};
 use crate::{fn_builder, service};
 
 use unic_langid::CharacterDirection;
@@ -154,19 +155,27 @@ impl Page {
     // Page RENDER.
 
     pub fn render(&mut self) -> ResultPage<Markup, FatalError> {
-        // Acciones de los módulos antes de renderizar la página.
+        // Acciones de los módulos antes de preparar la página.
-        run_actions(ACTION_BEFORE_RENDER_PAGE, |action| {
+        run_actions(ACTION_BEFORE_PREPARE_PAGE, |action| {
-            action_ref::<ActionBeforeRenderPage>(&**action).run(self)
+            action_ref::<ActionBeforePreparePage>(&**action).run(self)
         });
 
-        // Acciones del tema antes de renderizar la página.
+        // Acciones del tema antes de preparar la página.
-        self.context.theme().before_render_page(self);
+        self.context.theme().before_prepare_page(self);
 
-        // Primero, renderizar el cuerpo.
+        // Primero, preparar el cuerpo.
-        let body = self.context.theme().render_page_body(self);
+        let body = self.context.theme().prepare_page_body(self);
 
-        // Luego, renderizar la cabecera.
+        // Luego, preparar la cabecera.
-        let head = self.context.theme().render_page_head(self);
+        let head = self.context.theme().prepare_page_head(self);
+
+        // Acciones de los módulos después de preparar la página.
+        run_actions(ACTION_AFTER_PREPARE_PAGE, |action| {
+            action_ref::<ActionAfterPreparePage>(&**action).run(self)
+        });
+
+        // Acciones del tema después de preparar la página.
+        self.context.theme().after_prepare_page(self);
 
         // Finalmente, renderizar la página.
         let lang = self.langid().language.as_str();
@@ -183,7 +192,7 @@ impl Page {
         })
     }
 
-    pub fn render_region(&mut self, region: &str) -> Option<Markup> {
+    pub fn prepare_region(&mut self, region: &str) -> Option<Markup> {
         let render = self
             .regions
             .get_extended_bundle(self.context.theme().single_name(), region)