Skip to content

Add REST save for dashboard notification settings #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
altis-assistant wants to merge 1 commit into
base: master
Choose a base branch
from
+117 −3
Open

Add REST save for dashboard notification settings #71

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions library/Falcon.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ public static function bootstrap() {
if (is_admin()) {
Falcon_Admin::bootstrap();
}
else {
Falcon_Manager::bootstrap();
}

try {
// Check for a handler first
Expand Down
117 changes: 114 additions & 3 deletions library/Falcon/Manager.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,12 +122,123 @@ public static function register_dashboard_widget( $widgets ) {
wp_add_dashboard_widget( 'falcon_notification_settings', __( 'Notification Settings', 'falcon' ), array( get_class(), 'output_dashboard_widget' ) );
}

/**
* @wp-action rest_api_init
*/
public static function register_rest_routes() {
register_rest_route( 'falcon/v1', '/notification-settings', array(
'methods' => 'POST',
'callback' => array( get_class(), 'update_current_user_notification_settings' ),
'permission_callback' => array( get_class(), 'can_update_current_user_notification_settings' ),
'args' => array(
'settings' => array(
'required' => true,
'type' => 'object',
),
),
) );
}

public static function can_update_current_user_notification_settings() {
return is_user_logged_in() && current_user_can( 'read' ) && Falcon::is_enabled_for_site();
}

public static function update_current_user_notification_settings( WP_REST_Request $request ) {
$settings = $request->get_param( 'settings' );

if ( ! is_array( $settings ) ) {
return new WP_Error(
'falcon_invalid_notification_settings',
__( 'Notification settings must be an object.', 'falcon' ),
array( 'status' => 400 )
);
}

$args = array();
foreach ( $settings as $key => $value ) {
if ( ! is_scalar( $value ) ) {
continue;
}

$key = sanitize_text_field( wp_unslash( $key ) );
$value = sanitize_text_field( wp_unslash( $value ) );

// Falcon_Connector::save_profile_settings() expects the same mangled
// field names PHP creates for normal form submissions.
$args[ str_replace( '.', '_', $key ) ] = $value;
Comment on lines +166 to +168
Copy link
Copy Markdown

@svandragt svandragt May 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds like a helper method might be useful to codify the convention/

}

do_action( 'falcon.manager.save_profile_fields', get_current_user_id(), $args );

return rest_ensure_response( array(
'success' => true,
) );
}

public static function output_dashboard_widget() {
$user = wp_get_current_user();
?>
<table class="form-table">
<?php do_action( 'falcon.manager.profile_fields', $user ) ?>
</table>
<form id="falcon-notification-settings-form">
<table class="form-table">
<?php do_action( 'falcon.manager.profile_fields', $user ) ?>
</table>

<p>
<button type="submit" class="button button-primary"><?php esc_html_e( 'Save notification settings', 'falcon' ); ?></button>
<span class="spinner" aria-hidden="true"></span>
<span class="falcon-notification-settings-status" role="status" aria-live="polite"></span>
</p>
</form>

<script>
(function () {
var form = document.getElementById('falcon-notification-settings-form');
if (!form || !window.fetch || !window.FormData) {
return;
}

var button = form.querySelector('button[type="submit"]');
var spinner = form.querySelector('.spinner');
var status = form.querySelector('.falcon-notification-settings-status');

form.addEventListener('submit', function (event) {
event.preventDefault();

var settings = {};
var data = new FormData(form);
data.forEach(function (value, key) {
settings[key] = value;
});

button.disabled = true;
spinner.classList.add('is-active');
status.textContent = <?php echo wp_json_encode( __( 'Saving…', 'falcon' ) ); ?>;

fetch(<?php echo wp_json_encode( rest_url( 'falcon/v1/notification-settings' ) ); ?>, {
method: 'POST',
credentials: 'same-origin',
headers: {
'Content-Type': 'application/json',
'X-WP-Nonce': <?php echo wp_json_encode( wp_create_nonce( 'wp_rest' ) ); ?>
},
body: JSON.stringify({ settings: settings })
}).then(function (response) {
if (!response.ok) {
throw new Error(response.statusText || 'Request failed');
}

return response.json();
}).then(function () {
status.textContent = <?php echo wp_json_encode( __( 'Settings saved.', 'falcon' ) ); ?>;
}).catch(function () {
status.textContent = <?php echo wp_json_encode( __( 'Could not save settings.', 'falcon' ) ); ?>;
}).then(function () {
button.disabled = false;
spinner.classList.remove('is-active');
});
});
}());
</script>
<?php
}

Expand Down