bracketspace / notification Goto Github PK

In the wordpress dashboard after I installed this plugin, the ACF JS doesn't work anymore, it make impossible to edit fields and mange the content properly

Suggested by nepali65 at https://wordpress.org/support/topic/notify-on-published-post-update/

Example: email notification to Network Admin each time a new post is created on any site in the network

Catch exceptions and not die if WP_DEBUG is off

Hi Kuba,

I have issue when include Notifications from my pluign folder.

https://monosnap.com/file/9lWCdCtr5s85XMJ325jkqyRGfrBdpZ

Problem happens because of
add_filter( 'plugin_action_links', array( $this, 'plugins_table_link' ), 10, 5 );

I think need add condition for activate this only if plugin loaded standartalone.

Max

Here https://github.com/Kubitomakita/Notification/blob/dev/inc/Notification.php#L240

Thanks for reporting to @sebastianpisula

I am using Notifications to send email notifications on the creation and publication of a custom post type. And it's pretty awesome. But I am running into an issue with a custom created user.

The custom post type can be created by two types of users: Administrators and a Custom User.

I have two notifications set up:
1.) Alert by email when a new post has been submitted for review and
2.) Alert post creator when their post has been published.

Both alerts work if the custom post creator is logged in as Administrator - emails are received for both alerts: submit for review and publish. But if the posts are created by a custom user the first alert does not work. They do receive the 2nd alert (when their content has been published).

Do I need to adjust settings somewhere to recognize the custom user?

Any guidance is much appreciated. Thank you.

Feature suggested by @Pranick

Maybe as paid addon solution?

On the user profile page when I added notifications for new comments in the Disable box, I still received the notifications, even after clearing all caches. Using latest version of WP 4.7.4.

When new addon will be public, remove bbPress post types from settings

This happens when no comment types have been choosen in the settings page.

Basically the same happens also if the Post Type Option is empty.

This seems to be pretty useless and would be rather confusing.

When adding merge tag as a link TinyMCE is doing http://{tag} link.

In some cases it would be useful to have the option to use SMTP instead of wp_mail().

Would that be possible / wanted ?

When choosing new trigger when Text editor is active instead of the Visual editor js returns an error.

Would be good to add some contextual help, like the one you can see after installing Yoast SEO.

Prevent fatal errors when WP_DEBUG is inactive

I think will be very useful have some additional trigger description that will be shown when it was selected.

CC @max-kk

I have a notification with the trigger "Post published", but get constantly an email. Once the draft, or every time the post/draft is saved, i become a email notification. Not only when the post was published.

Current WordPress- and Plugin-Version

Sorry for my bad english

Generated plaintext from the HTML content

Feature proposed by Jeff Lehman

Get rid of the global variable by adding general action for each type and then just call the specific function and pass the type parameter.

Feature suggested by @JoeHana

Just like triggers for post types

Default recipient should always be there.

Let me state that I love this plugin, however, I noticed as of late that when this is enabled, I am unable to save a post or page as private. I've gone through every plugin and when I disabled this, I regained the ability. Testing this several time, resulted in the same result.

I'd love to help correct this, if this wasn't already reported. I've used this on wordpress 4.7, with plugin versions 2.4 and 3.0.

Thank you

ACF 5.5.11 saves the fields later than trigger is sent

I am trying to add a notification for my custom post type called 'message', but it isn't showing in the post types list on the settings page. Only Page and Post are showing but i have a few other custom post types as well. Any ideas why? Thanks

Hi Kuba,

I was tried to set the default "recipients" but can't figure out correct way to do this.

First example (admin notification):

$triggers['fv/competitor/to-admin/uploaded'] = array(
    'slug'     => 'fv/competitor/to-admin/uploaded',
    'name'     => __( 'New competitor submitted', 'fv' ),
    'group'    => '****(to admin)',
    'template' => $this->_get_notification_template('fv/competitor/to-admin/uploaded'),
    'tags'     => $this->_get_admin_default_tags(),
    'recipients' => array(
        'recipient_type_slug' => 'administrator',
        //'administrator'       => true,
    )
);

Problem: nothing happens

Second example (default - merge tag):

$triggers['fv/competitor/to-user/uploaded'] = array(
    'slug'     => 'fv/competitor/to-user/uploaded',
    'name'     => __( 'Competitor successful submitted', 'fv' ),
    'group'    => '*** (to user)',
    'template' => $this->_get_notification_template('fv/competitor/to-user/uploaded'),
    'tags'     => $this->_get_user_default_tags(),
    'recipients' => array(
        //'recipient_type_slug' => 'merge_tag',
        'merge_tag'           => 'competitor_user_email',
    )
);

Problem: by default Merge Tag type is selected but no one Merge Tag is selected and list is empty (example)

Thanks

Hi Kuba,

How about adding some conditial tags like IF?

There is a working example:

   
add_action( 'notification/notify/pre/message', 'pre_message__filter3', 10, 3 );

function pre_message__filter3($message, $trigger, $tags) {
            $matches = array();

            preg_match_all('/\[IF \{([^\}]*)\}\](.[^\]]+)(?:\[ELSE\](.+?))?\[ENDIF\]/s', $message, $matches);

            if ( empty($matches) ) {
                return $message;
            }

            $math_tag = '';
            foreach ( $matches[0] as $m_index => $match )
            {
                $math_tag =  trim($matches[1][$m_index]);

                if ( !empty($tags[$math_tag]) ) {
                    // IF value is not empty
                    $message = str_replace($match, $matches[2][$m_index], $message);
                } elseif( empty($tags[$math_tag]) && $matches[3][$m_index] ) {
                    // ELSE
                    $message = str_replace($match, $matches[3][$m_index], $message);
                } else {
                    // IF NO ELSE condition - REMOVE ALL
                    $message = str_replace($match, '', $message);
                }
            }

            return $message;
}

Example:

[IF {post_content}]Post content is filled![ENDIF]
[IF {post_content}]Post content is filled![ELSE]{post_content}[ENDIF]

Regards, Max

Taxonomies etc.

Allow easy way to get rid of all core triggers at once to make this plugin more like a framework than end product

I think there should be a way to translate (or describe) merge tags. This would be a huge help for non english users.

Something like is_notification_defined which will return boolean

Pass author ID to 3rd parameter of notification() function.
Also would be good to add current user ID

Hi - Is it possible to include a link to approve the comment in the WP Admin similar to how the default notifications work? Example:

Approve it: http://blog.achievers.com/wp-admin/comment.php?action=approve&c=372165#wpbody-content
Trash it: http://blog.achievers.com/wp-admin/comment.php?action=trash&c=372165#wpbody-content
Spam it: http://blog.achievers.com/wp-admin/comment.php?action=spam&c=372165#wpbody-content

Would email all users within specific role

I notice when I send an email that the NextGen Gallery code is sent along with the post_content. Is there a way to disable this?
[ngg_images source="galleries" container_ids="270" display_type="photocrati-nextgen_pro_masonry" size="180" padding="10" display_type_view="default" ngg_triggers_display="always" captions_enabled="0" captions_display_sharing="1" captions_display_title="1" captions_display_description="1" captions_animation="slideup" order_by="sortorder" order_direction="ASC" returns="included" maximum_entity_count="300"]

Thank you.

There should be an option to send comment notification AFTER it's processed by Akismet filters.

Feature proposed by Nels Johnson

when i activate the plugin, it breaks wordpress with error 500.. any idea what is wrong? would love to use this plugin! thanks