Title: HTML issues fixed (patch attached)
Last modified: August 21, 2016

---

# HTML issues fixed (patch attached)

 *  Resolved [ElementGreen](https://wordpress.org/support/users/elementgreen/)
 * (@elementgreen)
 * [11 years, 11 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/)
 * Not sure the best way to submit a patch, since I don’t see this plugin listed
   in the Trac issue database, so I opted to send it here. Let me know if there 
   is a more appropriate method. Here is a patch which fixes 3 HTML related formatting
   issues that break my theme on pages containing Buddypress Link interfaces.
 * * Added semicolon to &nbsp entity
    * Removed ‘/’ from option opening tags which
   have closing tags * Moved closing div tags to proper place (was breaking my sidebar)
 *     ```
       diff -ruN buddypress-links.orig/bp-links-templatetags.php buddypress-links/bp-links-templatetags.php
       --- buddypress-links.orig/bp-links-templatetags.php	2014-05-28 11:13:49.000000000 -0700
       +++ buddypress-links/bp-links-templatetags.php	2014-05-26 23:39:42.000000000 -0700
       @@ -767,7 +767,7 @@
        		$from_num = intval( ( $links_template->pag_page - 1 ) * $links_template->pag_num ) + 1;
        		$to_num = ( $from_num + ( $links_template->pag_num - 1 ) > $links_template->total_link_count ) ? $links_template->total_link_count : $from_num + ( $links_template->pag_num - 1) ;
   
       -		return sprintf( __( 'Viewing link %1$d to %2$d (of %3$d links)', 'buddypress-links' ), $from_num, $to_num, $links_template->total_link_count ) . '&nbsp<span class="ajax-loader"></span>';
       +		return sprintf( __( 'Viewing link %1$d to %2$d (of %3$d links)', 'buddypress-links' ), $from_num, $to_num, $links_template->total_link_count ) . '&nbsp;<span class="ajax-loader"></span>';
        	}
   
        function bp_links_total_link_count() {
       @@ -1433,7 +1433,7 @@
        		// is this one selected?
        		$selected = ( $selected_category_id == $category->id ) ? ' selected="selected"' : null;
        		// output it
       -		echo sprintf( '<option value="%d"%s />%s</option>', $category->id, $selected, $category->name ) . PHP_EOL;
       +		echo sprintf( '<option value="%d"%s>%s</option>', $category->id, $selected, $category->name ) . PHP_EOL;
        	}
   
        	do_action( 'bp_after_links_category_select_options' );
       diff -ruN buddypress-links.orig/themes/bp-links-default/index.php buddypress-links/themes/bp-links-default/index.php
       --- buddypress-links.orig/themes/bp-links-default/index.php	2014-05-28 11:13:49.000000000 -0700
       +++ buddypress-links/themes/bp-links-default/index.php	2014-05-27 00:08:45.000000000 -0700
       @@ -47,13 +47,11 @@
   
        		<?php do_action( 'bp_after_directory_links_content' ) ?>
   
       -		</div><!-- .padder -->
       -	</div><!-- #content -->
       -
        <?php
        	if ( current_theme_supports( 'buddypress' ) ):
        		// close old containers ?>
       -		</div></div><?php
       +			</div><!-- .padder -->
       +		</div><!-- #content --><?php
        		get_sidebar( 'buddypress' );
        		get_footer( 'buddypress' );
        	else:
       ```
   
 * [https://wordpress.org/plugins/buddypress-links/](https://wordpress.org/plugins/buddypress-links/)

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

 *  [VentureCore](https://wordpress.org/support/users/manakio2k/)
 * (@manakio2k)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959590)
 * Hey EG,
 * This is awesome, I can’t wait to try it out. I was basically having the same 
   issues so I removed it from my site.
 * Stupid Question #1
    Where do you put this?
 * Thanks!
 *  Thread Starter [ElementGreen](https://wordpress.org/support/users/elementgreen/)
 * (@elementgreen)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959591)
 * Its a patch file – [http://en.wikipedia.org/wiki/Patch_(Unix)](http://en.wikipedia.org/wiki/Patch_(Unix))
   You need to apply it to the existing sources. If you don’t have a patch program
   or know how to use it, you could apply the lines of text manually, since there
   aren’t that many changes and it is only in 2 files (buddypress-links/bp-links-
   templatetags.php and buddypress-links/themes/bp-links-default/index.php). Any
   lines that begin with the ‘-‘ symbol should be removed and lines with ‘+’ should
   be added (copy and paste them at the proper position in the file and remove the
   + symbol). You may want to make backups of the files first (although you could
   just re-install the plugin). Keep in mind that plugin upgrades will likely overwrite
   the changes, but hopefully the plugin author will fix these issues in the next
   release, so it wouldn’t matter then.
 *  [mrjarbenne](https://wordpress.org/support/users/mrjarbenne/)
 * (@mrjarbenne)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959592)
 * [@elementgreen](https://wordpress.org/support/users/elementgreen/) Thanks for
   doing the leg work on this. Worked out perfectly and fixed the issues I have 
   having today implementing this plugin.
 * Cheers.
 *  Thread Starter [ElementGreen](https://wordpress.org/support/users/elementgreen/)
 * (@elementgreen)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959593)
 * [@mrjarbenne](https://wordpress.org/support/users/mrjarbenne/) You’re welcome.
   I’m glad my efforts were helpful to you.
 * Cheers.
 *  [VentureCore](https://wordpress.org/support/users/manakio2k/)
 * (@manakio2k)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959594)
 * Hey EG, I just wanted to let you know I finally have it up on my site and looking
   good…
 * Now my issue is this 🙁
 * The BP Links menu item isn’t showing up on my admin menu so I can’t add/edit 
   categories or access links from admin panel.
 * Have you heard of or seen this problem before?
 * Any thoughts?
 * You can see the site here 🙂
    [http://seemespace.com/links](http://seemespace.com/links)
 *  [mrjarbenne](https://wordpress.org/support/users/mrjarbenne/)
 * (@mrjarbenne)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959595)
 * [@manakio2k](https://wordpress.org/support/users/manakio2k/) you should probably
   create a separate forum thread for this issue as it is most likely unrelated.
   It would be helpful if you could include the version of WP and BP in that separate
   post, and whether or not your site is a multisite, and if so, is the plugin activated
   network wide or just on the main domain.
 *  [VentureCore](https://wordpress.org/support/users/manakio2k/)
 * (@manakio2k)
 * [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959596)
 * Comments moved to new thread, thanks!

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

The topic ‘HTML issues fixed (patch attached)’ is closed to new replies.

 * ![](https://s.w.org/plugins/geopattern-icon/buddypress-links_d4d4d4.svg)
 * [BuddyPress Links](https://wordpress.org/plugins/buddypress-links/)
 * [Frequently Asked Questions](https://wordpress.org/plugins/buddypress-links/#faq)
 * [Support Threads](https://wordpress.org/support/plugin/buddypress-links/)
 * [Active Topics](https://wordpress.org/support/plugin/buddypress-links/active/)
 * [Unresolved Topics](https://wordpress.org/support/plugin/buddypress-links/unresolved/)
 * [Reviews](https://wordpress.org/support/plugin/buddypress-links/reviews/)

 * 7 replies
 * 3 participants
 * Last reply from: [VentureCore](https://wordpress.org/support/users/manakio2k/)
 * Last activity: [11 years, 9 months ago](https://wordpress.org/support/topic/html-issues-fixed-patch-attached/#post-4959596)
 * Status: resolved