[buddypress-trac] [BuddyPress] #997: New theme structure doesn't work with parent theme not being a BP framework

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Sat Sep 19 16:53:41 UTC 2009


#997: New theme structure doesn't work with parent theme not being a BP framework
------------------------+---------------------------------------------------
  Reporter:  Detective  |       Owner:  Detective               
      Type:  defect     |      Status:  assigned                
  Priority:  major      |   Milestone:  1.1                     
Resolution:             |    Keywords:  has-patch, needs-testing
------------------------+---------------------------------------------------

Comment(by Jason_JM):

 Ok. Testing now. Should be done in a bit.  I have about 30min from now to
 test, so I'll post up what I've tested.

 I don't want you to think i'm being harsh or anything.  We're all helping
 out, this is good stuff.  One step at a time, we're all here to help to
 make that lean mean buddy pressing machine. oh ya!

 Concerning the conventions, BP may not fully be safe checking everything
 and I'm sorry that I've used that as an example. The only perfect code
 I've ever seen was Windows Vista RC1.  JUST KIDDING!  But It's more of a
 type-safety issue.

 Though in the code it looks like we always have control over things like
 ($message) you really never know what can pop in there.  Run-time errors,
 security issues, etc.   With starting out using the strict string '' we
 are assured an object to pivot the comparison with.  So if $message say
 has some nasty garbage, it will always fail.  No need for any exception
 monitoring, etc.  Now the likely hood of $message really ever having
 garbage? Who knows. But it's the practice that we should use, so when
 there are times it may matter things won't be so bad.  Hopefully...

-- 
Ticket URL: <http://trac.buddypress.org/ticket/997#comment:11>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list