Ticket #4072 (closed defect: wontfix)

Opened 2 years ago

Last modified 1 year ago

is_tag and get_tag_link clash with similarly named bbPress functions

Reported by: filosofo Assigned to: anonymous
Priority: high Milestone:
Component: Administration Version:
Severity: major Keywords: has-patch is_tag get_tag_link bbpress
Cc:

Description

[5149] introduced functions is_tag and get_tag_link, which are also bbPress functions, messing up any WP-bbPress integrations.

My patch changes is_tag to is_wp_tag and get_tag_link to get_wp_tag_link.

Attachments

tag.patch (3.6 kB) - added by filosofo on 04/03/07 02:41:07.

Change History

04/03/07 02:41:07 changed by filosofo

  • attachment tag.patch added.

(follow-up: ↓ 2 ) 04/03/07 12:35:14 changed by matt

Generally WP takes precedence in things like this, since bbPress' userbase is so much smaller. get_link isn't bad but is_wp_tag is ugly.

(in reply to: ↑ 1 ) 04/03/07 16:11:36 changed by filosofo

Replying to matt:

Generally WP takes precedence in things like this, since bbPress' userbase is so much smaller. get_link isn't bad but is_wp_tag is ugly.

Okay, my suggested names are ugly. But if we keep the WP names as is, then down the road someone's going to have to think of alternate non-ugly names for the bbPress functions.

I'm just suggesting that we expend the creative re-naming effort now, before these functions have officially been released in WP, and before bbPress users lose their existing template functions and wonder what happened.

04/14/07 16:22:07 changed by rob1n

  • priority changed from normal to high.

04/19/07 04:00:08 changed by spencerp

Yes, I agree with filosofo. (No offense of course!) This was just one of the issues folks including me were having when running UTW and bbPress together before sigh.

Reference links about it: http://www.neato.co.nz/archives/2007/02/04/ultimate-tag-warrior-314159265/ "The thing where bbPress and UTW wouldn’t play nice because UTW had a is_tag function, and bbPress had one too. [If UTW sees that the is_tag function already exists, it isn’t declared; and there’s a is_utwtag function that bbPress users can use instead. Yeah.]"

I posted this comment on her blog previous to that UTW version even: "spencerp January 22nd, 2007

Hi Christine,

Would it be possible to fix the compatibility issue between bbPress and UTW? Here’s a link to the thread at the bbPress support forums.

Quoted traveljunky’s post: “this bug is caused by the function is_tag(), which both utw and bbpress use, so what i just did right now was search through the utw-plugin-folder with dreamweaver for that tag, found it three times and changed it to is_utwtag(). no idea if i can just do that without problems appearing later but for now it seems to work.”

Through the file(s) changing this: is_tag() to: is_utwtag() seems to help those running bbPress and WordPress running the UTW plugin. I was just was wondering.. ;):) "

Here's the bbPress post about it as well: http://bbpress.org/forums/topic/78?replies=5#post-2374

04/20/07 04:01:35 changed by rob1n

  • milestone changed from 2.2 to 2.3.

Tags are in 2.3 now.

09/13/07 02:08:30 changed by ryan

  • status changed from new to closed.
  • resolution set to wontfix.

bbPress will be doing some changes to accommodate.

09/13/07 21:37:50 changed by Nazgul

  • milestone deleted.