[@shanebp Moderator 12 years, 2 months ago]

It’s the jquery in global.js

The ‘target’ becomes the icon rather than the a tag.
So if ( target.hasClass('delete-activity') fails.

Changing to this gets the right target:
if ( target.hasClass('icon-trash') {

And then you need to change:
var link_href = target.attr('href');

to:
var link_href = target.closest("a").attr("href");

otherwise the nonce won’t be found and the ajax will fail.

Note: why is global.js being loaded from BuddyPress Default Theme when BP Legacy Theme is being used? Don’t know, but something is funky in this particular client’s installation.

[@shanebp Moderator 12 years, 2 months ago]

Rather than filter it, prevent the entry from being created.

Try this in your theme/functions.php or plugins/bp-custom.php
It won’t remove existing entries, but should stop new ones.

function victor_dont_save_join_group_activity( $activity_object ) {
 
    $exclude = array( 'joined_group' );
 
    if( in_array( $activity_object->type, $exclude ) )
        $activity_object->type = false;
 
}
add_action('bp_activity_before_save', 'victor_dont_save_join_group_activity', 1, 1 );

[@shanebp Moderator 12 years, 2 months ago]

This issue gets stranger the farther I dig…

1. We aren’t using the BP Default theme, so why is the activity stream using global.js instead of buddypress.js from the legacy theme?

2. in either /js, this click handler is never triggered if an image is used in the < a > tag for activity->delete or activity->fav

jq('div.activity').click( function(event) {

[@shanebp Moderator 12 years, 2 months ago]

Always grep in the BP plugin folder first.

bp_group_create_button()
bp-groups\bp-groups-template.php ~L. 1890

[@shanebp Moderator 12 years, 2 months ago]

Replacing the text with a basic img tag has the same effect; when clicked, the activity item is deleted but on a page reload instead of via ajax.

So I don’t think it’s specific to font awesome.

[@shanebp Moderator 12 years, 3 months ago]

format got messed up, so look here for a non-jQuery approach: http://pastebin.com/0afNVh3e

[@shanebp Moderator 12 years, 3 months ago]

read this

[@shanebp Moderator 12 years, 3 months ago]

javascript:

function checkboxlimit(checkgroup, limit){
	for (var i=0; i<checkgroup.length; i++){
		checkgroup[i].onclick=function(){
		var checkedcount=0
		for (var i=0; i<checkgroup.length; i++)
			checkedcount+=(checkgroup[i].checked)? 1 : 0
		if (checkedcount>limit){
			alert("You can select a maximum of "+limit+" boxes.")
			this.checked=false
			}
		}
	}
}		
checkboxlimit(document.forms.your_form_name_or_id['field_name[]'], 5);

[@shanebp Moderator 12 years, 3 months ago]

>In the original function, there’s no return statement only echo.

The original function is not a filter hooked function.
echo $tab; is not a return.
Read up on WordPress functions, actions and filters.

[@shanebp Moderator 12 years, 3 months ago]

I’m surprised your code works.
Filter should always return something.

This is unfinished but outlines another approach, probably more ‘correct’:

function custom_bp_profile_group_tabs( $tabs, $groups, $group_name ) {

	tab_names = array('Base', 'GroupProfile' ); // add to as necessary
	
	foreach ( (array) $tabs as $tab ) {
	   // loop thru tab_names using something like strpos() to check for matches
           // if you find a match, use str_replace to add 'Edit '
		
	}
	
	return $tabs;
}
add_filter('xprofile_filter_profile_group_tabs','custom_bp_profile_group_tabs', 1, 3);

[@shanebp Moderator 12 years, 3 months ago]

In bp_profile_group_tabs, there is a filter hook:

$tabs = apply_filters( 'xprofile_filter_profile_group_tabs', $tabs, $groups, $group_name );

Write a filter function and do a string search & replace for each tab.
ex. Search for ‘Base’ and replace it with ‘Edit Base’.

[@shanebp Moderator 12 years, 3 months ago]

Might be related to your component, but probably just the usual cpt issues.

Do you have a theme template called single-music.php so that WP can display it?

Do you have a pre_get_posts filter in your theme/functions.php or plugin file so that WP knows you want a cpt?

// so music cpt is included in cat or tag calls
function stoi_query_post_type($query) {
  if(is_category() || is_tag()) {
    $post_type = get_query_var('post_type');
	if($post_type)
	    $post_type = $post_type;
	else
	    $post_type = array('post','music');
    $query->set('post_type',$post_type);
	return $query;
    }
}
add_filter('pre_get_posts', 'stoi_query_post_type');

[@shanebp Moderator 12 years, 3 months ago]

Try using the bp_pre_user_query_construct hook
codex example

[@shanebp Moderator 12 years, 3 months ago]

There are various plugins and services for turning a WP site into an app – google it.
Here’s one: appPresser

Remember that you don’t need to use WP/BP to call the database.
Often times, devs create apps that only use a subset of the data.
So rolling your own app is feasible and probably the best approach.

[@shanebp Moderator 12 years, 3 months ago]

The bbpress example is a good one.

But just fyi – the BuddyPress codex page is here: bp_notifications_add_notification page.

[@shanebp Moderator 12 years, 3 months ago]

That might be tricky to track down, given that ajax is involved.
Your best bet is to ask Headway.

[@shanebp Moderator 12 years, 3 months ago]

>where both want to display the posts hence the duplication

It’s not a display problem, although that’s how it manifests.

Two entries are being created when an update is created.
http://boyabouttown.net/members/jenifusion/activity/58/
http://boyabouttown.net/members/jenifusion/activity/57/

Did you switch to a WP theme ( 2013, etc ) and then create an activity update?

[@shanebp Moderator 12 years, 3 months ago]

Easy enough to test with some ‘bad’ data, but groups_update_groupmeta uses update_metadata which runs the value thru things like sanitize_meta, so you should be fine.

[@shanebp Moderator 12 years, 3 months ago]

It looks fine and is usable.
I like the stripped down listings on pages like ‘members’

But the only opinions that count are those of your users.

[@shanebp Moderator 12 years, 3 months ago]

They aren’t visible because they have no last_activity field.
Going forward, this should add it for new members – untested, goes in theme/functions.php

function pilon_add_timestamp( $user_id ) {
    bp_update_user_last_activity( $user_id );
}
add_action('user_register', 'pilon_add_timestamp', 10 , 1);

For existing members with no last_activity, you could write a script that gathers those member ids and then run them thru the function above.

[@shanebp Moderator 12 years, 3 months ago]

Read this.

[@shanebp Moderator 12 years, 3 months ago]

Nice. And thanks for the link to the discussion.

[@shanebp Moderator 12 years, 3 months ago]

This is the file:
buddypress\bp-templates\bp-legacy\buddypress\activity\entry.php

Create a template overload before you change it.

[@shanebp Moderator 12 years, 3 months ago]

buddypress\bp-templates\bp-legacy\buddypress\activity\activity-loop.php

[@shanebp Moderator 12 years, 3 months ago]

You could use the approach in the codex page you’ve referenced.

Probably easier to use this hook from WP Job manager: ‘job_manager_save_job_listing’
In the function that you create for that hook, use bp_activity_add.