Ticket #40672: 40672.diff

src/wp-includes/js/wp-api.js

@@ -1171 +1171 @@
                                         routeName = 'me';
                                 }
 
+                                var initializeModel = function( attributes, options ) {
+                                        wp.api.WPApiBaseModel.prototype.initialize.call( this, attributes, options );
+
+                                        /**
+                                         * Determine if a model requires ?force=true to actually delete them.
+                                         */
+                                        if (
+                                                ! _.isEmpty(
+                                                        _.filter(
+                                                                this.endpoints,
+                                                                function( endpoint ) {
+                                                                        return (
+
+                                                                                // Does the method support DELETE?
+                                                                                'DELETE' === endpoint.methods[0] &&
+
+                                                                                // Exclude models that support trash (Post, Page).
+                                                                                (
+                                                                                        ! _.isUndefined( endpoint.args.force.description ) &&
+                                                                                        'Whether to bypass trash and force deletion.' !== endpoint.args.force.description
+                                                                                )
+                                                                        );
+                                                                }
+                                                        )
+                                                )
+                                        ) {
+                                                this.requireForceForDelete = true;
+                                        }
+                                };
+
                                 // If the model has a parent in its route, add that to its class name.
                                 if ( '' !== parentName && parentName !== routeName ) {
                                         modelClassName = wp.api.utils.capitalizeAndCamelCaseDashes( parentName ) + wp.api.utils.capitalizeAndCamelCaseDashes( routeName );
@@ -1203 +1233 @@
                                                 // Include the array of route methods for easy reference.
                                                 methods: modelRoute.route.methods,
 
-                                                initialize: function( attributes, options ) {
-                                                        wp.api.WPApiBaseModel.prototype.initialize.call( this, attributes, options );
-
-                                                        /**
-                                                         * Posts and pages support trashing, other types don't support a trash
-                                                         * and require that you pass ?force=true to actually delete them.
-                                                         *
-                                                         * @todo we should be getting trashability from the Schema, not hard coding types here.
-                                                         */
-                                                        if (
-                                                                'Posts' !== this.name &&
-                                                                'Pages' !== this.name &&
-                                                                _.includes( this.methods, 'DELETE' )
-                                                        ) {
-                                                                this.requireForceForDelete = true;
-                                                        }
-                                                }
+                                                // Include the array of route endpoints for easy reference.
+                                                endpoints: modelRoute.route.endpoints,
+
+                                                initialize: initializeModel
                                         } );
                                 } else {
 
@@ -1247 +1264 @@
                                                 name: modelClassName,
 
                                                 // Include the array of route methods for easy reference.
-                                                methods: modelRoute.route.methods
+                                                methods: modelRoute.route.methods,
+
+                                                // Include the array of route endpoints for easy reference.
+                                                endpoints: modelRoute.route.endpoints,
+
+                                                initialize: initializeModel
                                         } );
                                 }