Skip to:
Content
Pages
Categories
Search
Top
Bottom

Description Groups

Viewing 21 replies - 1 through 21 (of 21 total)

  • r-a-y
    Keymaster

    @r-a-y

    This is a known bug with the Nouveau template pack.

    Keep track of https://buddypress.trac.wordpress.org/ticket/7895

    In the meantime, you can use the Legacy template pack by navigating to “Settings > BuddyPress > Options” and selecting Legacy.


    edukedaventura
    Participant

    @edukedaventura

    agree. knows when it may be solved. I think the Nouveau template design is much more attractive


    edukedaventura
    Participant

    @edukedaventura

    Another problem that I find in the Nouveau template is that “Group Administrators” is not translated into Spanish.
    In the Legacy template pack if the translation works but the size is very big and ugly.
    Please, if you can solve these problems I thank you


    ipokkel
    Participant

    @ipokkel

    For a temporary workaround this worked for me:

    buddypress\bp-templates\bp-nouveau\buddypress\groups\single\cover-image-header.php

    line 62
    <?php echo esc_html( bp_nouveau_group_meta()->description ); ?>

    replace with:
    <?php echo bp_nouveau_group_meta()->description; ?>

    reference: https://codex.wordpress.org/Function_Reference/esc_html


    edukedaventura
    Participant

    @edukedaventura

    thank you very much, I’ll try it


    edukedaventura
    Participant

    @edukedaventura

    @ipokkel It works well. many thanks


    ipokkel
    Participant

    @ipokkel

    @edukedaventura Glad to hear my workaround worked for you 🙂


    damon18
    Participant

    @damon18

    Thanks @ipokkel worked for me. Hopefully this will get into the next release.


    ipokkel
    Participant

    @ipokkel

    Glad it worked 4 u @damon18


    damon18
    Participant

    @damon18

    @ipokkel I made the change directly in the Buddypress file

    /wp-content/plugins/buddypress/bp-templates/bp-nouveau/buddypress/groups/single/cover-image-header.php

    I wasn’t sure if the override system worked on this kind of thing, could I move that file (and any others that need modification) under my child theme like

    /wp-content/themes/my-theme/buddypress/bp-templates/bp-nouveau/buddypress/groups/single/cover-image-header.php

    I know that sort of over-ride works for theme files like page.php and single.php but does it work for everything?


    ipokkel
    Participant

    @ipokkel

    @damon18 to be honest, I’m actually overriding the nouveau theme from inside my child theme, like you’d like to do. I just sort of simplified the solution initially shared on this post for everyone since not everyone edits their themes or creates custom child themes for their projects.

    So yes you’ve got the right idea but the correct file location would be

    /wp-content/themes/my-theme/buddypress/groups/single/cover-image-header.php

    so it’s without the /buddypress/bp-templates/bp-nouveau/ section.

    Hope that helps 🙂


    damon18
    Participant

    @damon18

    Thank!


    lookingahead
    Participant

    @lookingahead

    @ipokkel unfortunately that line (line 62) does not exist, and it appears there’s been a change to that file since you posted your workaround.

    so where would i place that fix now? there appear to be several places that are similar to the code you mentioned, but nothing is exactly the code you mentioned…i think. i may have missed it if it is elsewhere; short on coffee today. 😀

    thoughts?

    here’s the code in that file:

    <?php
    /**
     * BuddyPress - Groups Header
     *
     * @since 3.0.0
     * @version 3.1.0
     */
    ?>
    
    <?php bp_get_template_part( 'groups/single/parts/header-item-actions' ); ?>
    
    <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    	<div id="item-header-avatar">
    		<a href="<?php echo esc_url( bp_get_group_permalink() ); ?>" class="bp-tooltip" data-bp-tooltip="<?php echo esc_attr( bp_get_group_name() ); ?>">
    
    			<?php bp_group_avatar(); ?>
    
    		</a>
    	</div><!-- #item-header-avatar -->
    <?php endif; ?>
    
    <div id="item-header-content">
    
    	<p class="highlight group-status"><strong><?php echo esc_html( bp_nouveau_group_meta()->status ); ?></strong></p>
    
    	<p class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>">
    		<?php
    		echo esc_html(
    			sprintf(
    				/* translators: %s = last activity timestamp (e.g. "active 1 hour ago") */
    				__( 'active %s', 'buddypress' ),
    				bp_get_group_last_active()
    			)
    		);
    		?>
    	</p>
    
    	<?php bp_nouveau_group_hook( 'before', 'header_meta' ); ?>
    
    	<?php if ( bp_nouveau_group_has_meta_extra() ) : ?>
    		<div class="item-meta">
    
    			<?php echo bp_nouveau_group_meta()->extra; ?>
    
    		</div><!-- .item-meta -->
    	<?php endif; ?>
    
    		<?php if ( ! bp_nouveau_groups_front_page_description() ) { ?>
    			<?php if ( bp_nouveau_group_meta()->description ) { ?>
    				<div class="group-description">
    					<?php echo bp_nouveau_group_meta()->description; ?>
    				</div><!-- //.group_description -->
    			<?php	} ?>
    		<?php } ?>
    
    </div><!-- #item-header-content -->
    
    <?php bp_nouveau_group_header_buttons(); ?>
    

    lookingahead
    Participant

    @lookingahead

    @ipokkel lol NEVERMIND….was looking at the wrong file.

    used your fix and it worked. 😀 THANK YOUUUUUU

    P.S.: going on a coffee run right now. lol it’s apparent that i neeeed it, haha….

    😀


    ipokkel
    Participant

    @ipokkel

    Glad to hear you got it working for your BP 🙂


    ipokkel
    Participant

    @ipokkel

    Just a quick note:

    I’ve searched through their tickets and this is being addressed and from what I can figure out doing a quick read through the fix should be released in 3.2.0.

    The fixes for the defect follows a slightly different approach so it’s worth a read-through if you would like to attempt using a different method. I unfortunately haven’t got time to test for now, so I’m just mentioning the alternative for those interested.

    https://buddypress.trac.wordpress.org/ticket/7923


    lookingahead
    Participant

    @lookingahead

    @ipokkel okay lol….i had to restore my site to an earlier time, so….this fix was lost. totally sucks.

    and, i updated to the latest version. which apparently came out after your fix was posted.

    so here’s what you recommended:

    For a temporary workaround this worked for me:

    buddypress\bp-templates\bp-nouveau\buddypress\groups\single\cover-image-header.php

    line 62
    <?php echo esc_html( bp_nouveau_group_meta()->description ); ?>

    replace with:
    <?php echo bp_nouveau_group_meta()->description; ?>

    reference: https://codex.wordpress.org/Function_Reference/esc_html

    so i interpreted that location as:
    wp-content\plugins\buddypress\bp-templates\bp-nouveau\buddypress\groups\single\cover-image-header.php

    but when i got there, there was no line 62 as there had been before. even after the coffee this time… 🙂

    here’s the contents of the cover-image-header PHP file that i see at that location now:

    <?php
    /**
     * BuddyPress - Groups Cover Image Header.
     *
     * @since 3.0.0
     * @version 3.2.0
     */
    ?>
    
    <div id="cover-image-container">
    	<div id="header-cover-image"></div>
    
    	<div id="item-header-cover-image">
    		<?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    			<div id="item-header-avatar">
    				<a href="<?php echo esc_url( bp_get_group_permalink() ); ?>" title="<?php echo esc_attr( bp_get_group_name() ); ?>">
    
    					<?php bp_group_avatar(); ?>
    
    				</a>
    			</div><!-- #item-header-avatar -->
    		<?php endif; ?>
    
    <?php	if ( ! bp_nouveau_groups_front_page_description() ) : ?>
    		<div id="item-header-content">
    
    			<p class="highlight group-status"><strong><?php echo esc_html( bp_nouveau_group_meta()->status ); ?></strong></p>
    			<p class="activity" data-livestamp="<?php bp_core_iso8601_date( bp_get_group_last_active( 0, array( 'relative' => false ) ) ); ?>">
    				<?php
    				/* translators: %s = last activity timestamp (e.g. "active 1 hour ago") */
    				printf( __( 'active %s', 'buddypress' ), bp_get_group_last_active() );
    				?>
    			</p>
    
    			<?php echo bp_nouveau_group_meta()->group_type_list; ?>
    			<?php bp_nouveau_group_hook( 'before', 'header_meta' ); ?>
    
    			<?php if ( bp_nouveau_group_has_meta_extra() ) : ?>
    				<div class="item-meta">
    
    					<?php echo bp_nouveau_group_meta()->extra; ?>
    
    				</div><!-- .item-meta -->
    			<?php endif; ?>
    
    			<?php bp_nouveau_group_header_buttons(); ?>
    
    		</div><!-- #item-header-content -->
    <?php endif; ?>
    
    		<?php bp_get_template_part( 'groups/single/parts/header-item-actions' ); ?>
    
    	</div><!-- #item-header-cover-image -->
    
    </div><!-- #cover-image-container -->
    
    <?php if ( ! bp_nouveau_groups_front_page_description() && bp_nouveau_group_has_meta( 'description' ) ) : ?>
    	<div class="desc-wrap">
    		<div class="group-description">
    			<?php bp_group_description(); ?>
    		</div><!-- //.group_description -->
    	</div>
    <?php endif; ?>

    where do i put that fix now?????

    thanks in advance for any and all help.


    ipokkel
    Participant

    @ipokkel

    This workaround was for Buddypress 3.1.0 and in 3.2.0 this issue (bug) was fixed by BP developers. This means if you’re using BP 3.2.0 you’re group description should display correctly without having to modify any of the core BP files or through using a template override in your theme/child-theme.

    I’ve upgraded my BP to 3.2.0 and removed my workaround and my group descriptions display perfectly.


    lookingahead
    Participant

    @lookingahead

    @ipokkel unfortunately lol upgrading to 3.2.0 didn’t help for my issue.

    it’s definitely 3.2.0 that i have. it’s the strangest thing. the workaround worked perfectly before; i’m almost tempted to reinstall the previous version, just to get that fix back.


    ipokkel
    Participant

    @ipokkel

    @lookingahead can you give me a link to the page where you’re having this problem.


    ipokkel
    Participant

    @ipokkel

    @lookingahead, I did a test with a fresh WP 4.9.8 and BP 3.2.0 and no issues.

    Pop me a mail at ipokkel at gmail and I’ll see if I can help you sort out the bp group display on your site.

Viewing 21 replies - 1 through 21 (of 21 total)
  • You must be logged in to reply to this topic.
Skip to toolbar