Bug #11243
openAudit bp-custom.php
Added by Raymond Hoh about 5 years ago. Updated almost 2 years ago.
0%
Description
Over the years, we've added a lot of code to our BuddyPress custom code file - bp-custom.php.
Some of that code is no longer applicable to the Commons today. It's time to trim the fat for 1.15!
Files
| BP_Error.PNG (4.72 KB) BP_Error.PNG | Michael Shields, 2019-04-23 08:47 PM |
Updated by Raymond Hoh about 5 years ago
I've done the following so far:
- Remove older references to bbPress v1 - https://github.com/cuny-academic-commons/cac/commit/a7e0f2a95724ce575e41d317f62469757164484b
- Remove oohEmbed - https://github.com/cuny-academic-commons/cac/commit/f24bf431472d14cb04a21d549a862e011ca83f84
- Remove 'foratv' shortcode - https://github.com/cuny-academic-commons/cac/commit/ce96febdc58f6af4fc3e1e5e41b1e9b04a55c4a1
- Remove older hotfixes that have been addressed in newer versions of BuddyPress - https://github.com/cuny-academic-commons/cac/commit/ce11f27ef887c1fd7e4961edf55e82aada6be18b
bbPress v1 is no longer in use on the Commons and both oohEmbed and ForaTV's websites no longer exist.
I'll do one more sweep of the bp-custom.php file in the next week or so.
Updated by Matt Gold about 5 years ago
I love this kind of housekeeping. Thank you for doing it!
Updated by Boone Gorges almost 5 years ago
- Target version changed from 1.15 to 1.16
Let's do another concerted sweep for 1.16.
Updated by Michael Shields almost 5 years ago
- File BP_Error.PNG BP_Error.PNG added
Hi guys,
I'm not sure if this bug is related to this thread but when I went to edit https://oirap.commons.gc.cuny.edu/applications-admissions/cas-freshman-application-data/, I saw the attached error message. When I clicked on the link, I received a "Sorry, you are not allowed to access this page." error message. Note that we do not use email functionality on that (or any) page on that site.
BTW, in the nearly 11 months since rolling out the OIRA Wiki to the larger CUNY community, everyone LOVES it! Thank you so much for all of your support last spring and summer as I worked to get things up and running. I couldn't have done it without this fantastic team!
Updated by Matt Gold almost 5 years ago
That's so fantastic to hear, Michael -- thank you for letting us know!!!
Updated by Boone Gorges almost 5 years ago
Thanks for reporting this, Michael. The issue you've noted is unrelated to the bp-custom.php changes in this ticket, but instead comes from some unrelated plugin updates in yesterday's maintenance release. I've made the necessary changes to ensure that you'll no longer see this notice. (See also https://github.com/boonebgorges/buddypress-group-email-subscription/issues/174 for the distribution plugin.)
Updated by Michael Shields almost 5 years ago
Thank you, Boone. I confirm that I'm no longer seeing that message.
Updated by Boone Gorges over 4 years ago
- Target version changed from 1.16 to 1.17.0
This is an ongoing task.
Updated by Boone Gorges almost 4 years ago
- Target version changed from 1.17.0 to 1.18.0
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.0 to 1.19.0
Updated by Boone Gorges over 2 years ago
- Target version changed from 1.19.0 to 2.0.0
Updated by Boone Gorges almost 2 years ago
- Target version changed from 2.0.0 to Future release