+
+
+
+
+
get_first_party_action_items();
}
+ // Lost submissions, outside the logging gate and on every screen: data is
+ // gone, and Distraction Free (which turns logging off) must not hide it.
+ $warnings = array_merge( $warnings, $this->get_lost_submissions_action_items() );
+
+ // Email delivery advice, outside the logging gate: it is about the site's
+ // setup, not about recorded failures. On the SureForms dashboard only --
+ // an owner who has not set up email yet needs telling where they manage
+ // forms, not on every admin screen -- which also keeps the form lookup
+ // behind it off every other page load.
+ $on_dashboard = Helper::validate_request_context( 'sureforms_menu', 'page' );
+ if ( $on_dashboard ) {
+ $warnings = array_merge( $warnings, $this->get_email_delivery_action_items() );
+ }
+
+ // A quota running out also belongs on Entries, where an owner notices
+ // submissions arriving but not their emails.
+ if ( $on_dashboard || Helper::validate_request_context( SRFM_ENTRIES, 'page' ) ) {
+ $quota_items = $this->get_email_quota_action_items();
+ $warnings = array_merge( $warnings, $quota_items );
+
+ // SureForms hides classic notices on its own screens and Entries has no
+ // Form Checks panel, so it gets the warning in its React notice bar. The
+ // bar reads the notices when the page data is filtered, after this.
+ if ( ! $on_dashboard ) {
+ foreach ( $quota_items as $quota_item ) {
+ Notice_Manager::register_notice(
+ [
+ 'id' => 'srfm-' . $quota_item['id'],
+ 'variant' => $quota_item['status'],
+ 'title' => $quota_item['title'],
+ 'message' => $quota_item['message'],
+ 'actions' => [
+ [
+ 'label' => $quota_item['cta_label'],
+ 'url' => $quota_item['cta_url'],
+ 'target' => '_blank',
+ 'variant' => 'link',
+ ],
+ ],
+ 'dismissible' => false,
+ 'pages' => [ SRFM_ENTRIES ],
+ ]
+ );
+ }
+ }
+ }
+
$this->track_action_item_impressions( $warnings );
/**
@@ -3719,8 +3854,8 @@ public function get_action_items() {
*/
$items = Helper::apply_filters_as_array( 'srfm_action_items', $warnings );
- // Both URLs normalised once, here, rather than trusting each renderer to do
- // it. Two things are being fixed at once.
+ // The CTA, guide and connect URLs normalised once, here, rather than
+ // trusting each renderer to do it. Two things are being fixed at once.
//
// The scheme: the classic notice runs esc_url() and drops anything outside
// the allowlist, while React assigns href directly and react-dom 18 leaves
@@ -3741,7 +3876,7 @@ public function get_action_items() {
continue;
}
- foreach ( [ 'cta_url', 'guide_url' ] as $key ) {
+ foreach ( [ 'cta_url', 'guide_url', 'connect_url' ] as $key ) {
if ( ! isset( $item[ $key ] ) ) {
continue;
}
@@ -3929,6 +4064,24 @@ private function get_first_party_action_items() {
],
];
+ // With SureMail sending, the reason for the failure is in its email log --
+ // a closer next step than a general guide.
+ if ( 'suremail' === Mailer::get_active()['slug'] ) {
+ $categories['notification']['guide'] = Suremail_Adapter::logs_url();
+ $categories['notification']['guide_label'] = __( 'View SureMail Logs', 'sureforms' );
+ $categories['notification']['guide_action'] = 'view_logs';
+ }
+
+ // A form that does not store entries has no saved copy to fall back on: the
+ // email that failed was the only record of that submission. "The entry was
+ // saved" would be false, and the fix differs, so say both.
+ if ( isset( $open['notification'] ) && false === ( $open['notification']['entry_saved'] ?? true ) ) {
+ /* translators: %s: form title. */
+ $categories['notification']['title'] = __( 'A notification failed on %s, and the submission was not stored.', 'sureforms' );
+ $categories['notification']['generic'] = __( 'A notification failed, and the submission was not stored.', 'sureforms' );
+ $categories['notification']['message'] = __( 'This form is set to never store entries, so the email was the only copy of that submission and it could not be sent. Set up email delivery, or turn off "Never store entry data after form submission" for this form, so submissions are not lost.', 'sureforms' );
+ }
+
foreach ( $categories as $category => $copy ) {
if ( ! isset( $open[ $category ] ) ) {
continue;
@@ -3970,9 +4123,15 @@ private function get_first_party_action_items() {
// without a guide needs no branch in either renderer, and neither does
// an item contributed through srfm_action_items.
if ( ! empty( $copy['guide'] ) ) {
- $warning['guide_label'] = __( 'Help Me Fix', 'sureforms' );
+ $warning['guide_label'] = $copy['guide_label'] ?? __( 'Help Me Fix', 'sureforms' );
$warning['guide_url'] = $copy['guide'];
- $warning['guide_action'] = 'help_me_fix';
+ $warning['guide_action'] = $copy['guide_action'] ?? 'help_me_fix';
+ }
+
+ // A failed notification on a site with no sending service behind it:
+ // lead with the one-click fix.
+ if ( 'notification' === $category ) {
+ $warning = array_merge( $warning, $this->get_connect_surecontact_action() );
}
$warnings[] = $warning;
@@ -4008,6 +4167,308 @@ private function get_first_party_action_items() {
return $warnings;
}
+ /**
+ * "Connect SureContact" for a notification failure, when nothing is sending.
+ *
+ * Offered while SureMail is not active, or is active with no connection --
+ * the cases where the failure is almost certainly the missing service. With
+ * a connection in place the failure is that connection's, and View SureMail
+ * Logs is the closer next step. Hidden with promotions.
+ *
+ * Leads to SureForms -> SMTP while SureMail is not active, which installs
+ * it; once active, straight to SureMail's SureContact screen. (The SMTP page
+ * hands over to that screen after install once #3205 is in.) The account
+ * itself is only ever created behind SureMail's consent.
+ *
+ * @since 2.12.9
+ * @return array