Ticket #8065: 8065.patch

src/bp-templates/bp-nouveau/buddypress/common/js-templates/activity/form.php

@@ -2 +2 @@
 /**
  * Activity Post form JS Templates
  *
- * @version 3.1.0
+ * @since 3.0.0
+ * @version 5.0.0
  */
 ?>
 
+<script type="text/html" id="tmpl-activity-before-post-form-inputs">
+        <?php bp_nouveau_activity_hook( 'before', 'post_form' ); ?>
+</script>
+
 <script type="text/html" id="tmpl-activity-post-form-feedback">
         <span class="bp-icon" aria-hidden="true"></span><p>{{{data.message}}}</p>
 </script>

src/bp-templates/bp-nouveau/includes/activity/functions.php

@@ -3 +3 @@
  * Activity functions
  *
  * @since 3.0.0
- * @version 3.1.0
+ * @version 5.0.0
  */
 
 // Exit if accessed directly.
@@ -67 +67 @@
         $activity_params = array(
                 'user_id'     => bp_loggedin_user_id(),
                 'object'      => 'user',
-                'backcompat'  => (bool) has_action( 'bp_activity_post_form_options' ),
+                'backcompat'  => array(
+                        'before_post_form'  => (bool) has_action( 'bp_before_activity_post_form' ),
+                        'post_form_options' => (bool) has_action( 'bp_activity_post_form_options' ),
+                ),
                 'post_nonce'  => wp_create_nonce( 'post_update', '_wpnonce_post_update' ),
         );
 

src/bp-templates/bp-nouveau/includes/activity/template-tags.php

@@ -3 +3 @@
  * Activity Template tags
  *
  * @since 3.0.0
- * @version 3.1.0
+ * @version 5.0.0
  */
 
 // Exit if accessed directly.
@@ -67 +67 @@
  * Enqueue needed scripts for the Activity Post Form
  *
  * @since 3.0.0
+ * @since 5.0.0 Move the `bp_before_activity_post_form` hook inside the Activity post form.
  */
 function bp_nouveau_before_activity_post_form() {
         if ( bp_nouveau_current_user_can( 'publish_activity' ) ) {
                 wp_enqueue_script( 'bp-nouveau-activity-post-form' );
         }
-
-        /**
-         * Fires before the activity post form.
-         *
-         * @since 1.2.0
-         */
-        do_action( 'bp_before_activity_post_form' );
 }
 
 /**

src/bp-templates/bp-nouveau/js/buddypress-activity-post-form.js

@@ -1 +1 @@
 /* global bp, BP_Nouveau, _, Backbone */
-/* @version 3.1.0 */
+/* @since 3.0.0 */
+/* @version 5.0.0 */
 window.wp = window.wp || {};
 window.bp = window.bp || {};
 
@@ -375 +376 @@
                 template : bp.template( 'activity-post-form-options' )
         } );
 
+        bp.Views.BeforeFormInputs = bp.View.extend( {
+                tagName  : 'div',
+                template : bp.template( 'activity-before-post-form-inputs' )
+        } );
+
         bp.Views.FormTarget = bp.View.extend( {
                 tagName   : 'div',
                 id        : 'whats-new-post-in-box',
@@ -586 +592 @@
                                 BP_Nouveau.activity.params,
                                 ['user_id', 'item_id', 'object' ]
                         ) );
+                        this.options.backcompat = BP_Nouveau.activity.params.backcompat;
+                        var staticViews = [
+                                new bp.Views.FormAvatar(),
+                                new bp.Views.FormContent( { activity: this.model } )
+                        ];
+
+                        // Backcompat to take the `bp_before_activity_post_form` action in account.
+                        if ( true === this.options.backcompat.before_post_form ) {
+                                staticViews.unshift( new bp.Views.BeforeFormInputs() );
+                        }
 
                         // Clone the model to set the resetted one
                         this.resetModel = this.model.clone();
 
-                        this.views.set( [
-                                new bp.Views.FormAvatar(),
-                                new bp.Views.FormContent( { activity: this.model } )
-                        ] );
+                        this.views.set( staticViews );
 
                         this.model.on( 'change:errors', this.displayFeedback, this );
                 },
 
                 displayFull: function( event ) {
+                        var numStaticViews = true === this.options.backcompat.before_post_form ? 3 : 2;
 
                         // Remove feedback.
                         this.cleanFeedback();
 
-                        if ( 2 !== this.views._views[''].length ) {
+                        if ( numStaticViews !== this.views._views[''].length ) {
                                 return;
                         }
 
@@ -612 +626 @@
                                 height : 'auto'
                         } );
 
-                        // Backcompat custom fields
-                        if ( true === BP_Nouveau.activity.params.backcompat ) {
+                        // Add the container view for buttons or custom fields.
+                        if ( true === this.options.backcompat.post_form_options ) {
                                 this.views.add( new bp.Views.FormOptions( { model: this.model } ) );
+                        } else {
+                                this.views.add( new bp.View( { id: 'whats-new-options' } ) );
                         }
 
                         // Attach buttons
                         if ( ! _.isUndefined( BP_Nouveau.activity.params.buttons ) ) {
                                 // Global
                                 bp.Nouveau.Activity.postForm.buttons.set( BP_Nouveau.activity.params.buttons );
-                                this.views.add( new bp.Views.FormButtons( { collection: bp.Nouveau.Activity.postForm.buttons, model: this.model } ) );
+                                this.views.add( '#whats-new-options', new bp.Views.FormButtons( { collection: bp.Nouveau.Activity.postForm.buttons, model: this.model } ) );
                         }
 
                         // Select box for the object
                         if ( ! _.isUndefined( BP_Nouveau.activity.params.objects ) && 1 < _.keys( BP_Nouveau.activity.params.objects ).length ) {
-                                this.views.add( new bp.Views.FormTarget( { model: this.model } ) );
+                                this.views.add( '#whats-new-options', new bp.Views.FormTarget( { model: this.model } ) );
                         }
 
-                        this.views.add( new bp.Views.FormSubmit( { model: this.model } ) );
+                        this.views.add( '#whats-new-options', new bp.Views.FormSubmit( { model: this.model } ) );
                 },
 
                 resetForm: function() {
+                        var self = this, indexStaticViews = self.options.backcompat.before_post_form ? 2 : 1;
+
                         _.each( this.views._views[''], function( view, index ) {
-                                if ( index > 1 ) {
+                                if ( index > indexStaticViews ) {
                                         view.remove();
                                 }
                         } );