Changeset 12110

Timestamp:

10/07/2022 07:42:52 AM (2 years ago)

Author:

coffee2code

Message:

Photo Directory, Uploads: Allow users with 30 or more published photos to bulk check the submission checkboxes.

Props topher1kenobe, coffee2code.
Fixes #6239.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory

Files:

• assets/js/submit.js (modified) (1 diff)
• inc/uploads.php (modified) (1 diff)
• inc/user.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory/assets/js/submit.js

@@ -11 +11 @@
     }
 
-    // Prevent double-submission of upload form.
+    // Add checkbox to set/unset all other confirmation checkboxes.
+    if ( PhotoDir.user_can_toggle_all_checkboxes ) {
+        const checkboxDiv = document.createElement( 'div' );
+        checkboxDiv.setAttribute('class', 'confirmation-checkbox-toggle-container');
+        const checkboxLabel = document.createElement( 'label' );
+        const checkboxToggle = document.createElement( 'input' );
+        checkboxDiv.insertAdjacentElement( 'afterbegin', checkboxLabel );
+        checkboxLabel.insertAdjacentElement( 'afterbegin', checkboxToggle );
+        checkboxToggle.setAttribute('class', 'confirmation-checkbox-toggle');
+        checkboxToggle.setAttribute('name', 'confirmation-checkbox-toggle');
+        checkboxToggle.setAttribute('type', 'checkbox');
+        checkboxLabel.insertAdjacentText( 'beforeend', ' ' + PhotoDir.toggle_all_checkboxes );
+        checkboxToggle.addEventListener( 'change', e => {
+            const newState = e.target.checked;
+            // For checkboxes not matching the current toggle state, click them so that they do.
+            document.querySelectorAll('.upload-checkbox-wrapper input[type="checkbox"]:not(.confirmation-checkbox-toggle)').forEach( v => {
+                if ( v.checked !== newState ) {
+                    v.click();
+                }
+            } );
+        } );
+        document.querySelector('.upload-checkbox-wrapper p').insertAdjacentElement( 'afterend', checkboxDiv );
+    }
+
     const photo_fieldset = document.getElementById('wporg-photo-upload');
     if ( null === photo_fieldset ) {

sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory/inc/uploads.php

@@ -285 +285 @@
                     'min_file_dimension' => self::get_minimum_photo_dimension(),
                     'msg_validating_dimensions' => __( 'Validating photo dimensions…', 'wporg-photos' ),
+
+                    // Toggle all confirmation checkboxes for upload.
+                    'user_can_toggle_all_checkboxes' => User::can_toggle_confirmation_checkboxes(),
+                    'toggle_all_checkboxes' => __( 'As a contributor of many photos, I am well aware of the requirements listed below and agree to them all.', 'wporg-photos' ),
                 ]
             );

sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory/inc/user.php

@@ -19 +19 @@
      */
     const MAX_PENDING_SUBMISSIONS = 5;
+
+    /**
+     * The number of published posts before a given user is permitted to toggle
+     * all of the confirmation checkboxes when submitting a photo.
+     *
+     * @var int
+     */
+    const TOGGLE_ALL_THRESHOLD = 30;
 
     public static function init() {
@@ -89 +97 @@
     }
 
+    /**
+     * Determines if a user is eligible to toggle all confirmation checkboxes on
+     * the photo upload form.
+     *
+     * The intent is that users who have submitted a decent number of published
+     * photos are as aware of the listed criteria as they're going to be and have
+     * demonstrated that they are able to abide by them.
+     *
+     * @todo Handle eventual case when a new checkbox is added or one is changed
+     *       enough to warrant making the user manually re-check the checkboxes.
+     *       The latest checkbox update date can be stored as a constant and, if
+     *       set, the contributor must also have a post (or N posts) published
+     *       after that date to requalify for the bulk toggle.
+     *
+     * @param int $user_id Optional. The user ID. If not defined, assumes current
+     *                     user. Default false.
+     * @return bool True if user can toggle confirmation checkboxes, else false.
+     */
+    public static function can_toggle_confirmation_checkboxes( $user_id = false ) {
+        $can = false;
+
+        if ( ! $user_id ) {
+            $user_id = get_current_user_id();
+        }
+
+        if ( $user_id && self::count_published_photos( $user_id ) >= self::TOGGLE_ALL_THRESHOLD ) {
+            $can = true;
+        }
+
+        return $can;
+    }
+
 }