CUNY Graduate Center - Project Tracking System: Issueshttps://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2024-03-11T22:29:12ZCUNY Graduate Center - Project Tracking System
Redmine CUNY Academic Commons - Bug #19944 (New): Dealing with BuddyPress 12.x upgrade, pt. 2https://redmine.gc.cuny.edu/issues/199442024-03-11T22:29:12ZRaymond Hoh
<p>Follow-up to <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Dealing with BuddyPress 12.x upgrade (Resolved)" href="https://redmine.gc.cuny.edu/issues/19493">#19493</a>. Just wanted to make sure my comments in <a class="external" href="https://redmine.gc.cuny.edu/issues/19493#note-10">https://redmine.gc.cuny.edu/issues/19493#note-10</a> are brought forward to a future release:</p>
<blockquote>
<p>For more pronounced changes, I created a new branch -- <a href="https://github.com/cuny-academic-commons/cac/compare/bp12...bp12-part2" class="external">bp12-part2</a> -- that namely switches out all BuddyPress 12 deprecated function calls. We can roll that out in a later maintenance release.</p>
<p>I've been testing <code>bp12-part2</code> with BP Classic <em>deactivated</em> and with <code>BP_IGNORE_DEPRECATED</code> set to <code>true</code> and these are some issues I encountered:</p>
<ul>
<li>BP Event Organiser needs some minor adjustments to account for group subnav registration: <a class="external" href="https://github.com/cuny-academic-commons/bp-event-organiser/compare/1.2.x...bp12#diff-b0397cc71e95a79af968e8dadb7b8e525085f462c5a88ac11c8bdb86ecd7173d">https://github.com/cuny-academic-commons/bp-event-organiser/compare/1.2.x...bp12#diff-b0397cc71e95a79af968e8dadb7b8e525085f462c5a88ac11c8bdb86ecd7173d</a></li>
<li>bbPress does <a href="https://github.com/bbpress/bbPress/blob/4178cbf665674c128c928fabdc0322badacad06b/src/includes/extend/buddypress/groups.php#L140-L157" class="external">some conditional checks</a> that needs to be delayed to the <code>'bp_parse_query'</code> hook. This fixes an issue with the bbPress stylesheet not loading on group forum pages I mentioned above in <a href="https://redmine.gc.cuny.edu/issues/19493#note-4" class="external">comment 4</a> amongst other permission-related cap overrides for the current user.</li>
<li>I need Git permissions to the <code>tapor-client</code> repo so I can push changes and pin a new version for our <code>cac</code> repo.</li>
</ul>
<p>About the bbPress load order issue:</p>
<blockquote>
<p>Furthermore, when running bp-classic, I had to do some additional work to ensure that bbPress's BP compatibility layer is loaded in time, and is loaded only once. Like the approach above, it's pretty hacky (requiring a static variable to prevent double-loading) but it basically works. Here's the full patch: <a class="external" href="https://github.com/openlab-at-city-tech/openlab/commit/27f2a68354550295d85f0f3c5aaafc55004268f0">https://github.com/openlab-at-city-tech/openlab/commit/27f2a68354550295d85f0f3c5aaafc55004268f0</a></p>
</blockquote>
<p>I tackled this a little differently in my bbPress commit. See <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-0f345898d2c7666be77acc95ec88e90e98a407644a4ce5431b3fb96d06749039">https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-0f345898d2c7666be77acc95ec88e90e98a407644a4ce5431b3fb96d06749039</a> and the constructor and includes() portion of <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-a8d1ad6e40805411d0e557ee641e9d47c6cc3d0f17ef107f22ac729255439a5a">https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-a8d1ad6e40805411d0e557ee641e9d47c6cc3d0f17ef107f22ac729255439a5a</a> . I think the constructor changes might fix your double load order issue. Can you do some tests and see if this works for you, Boone?</p>
</blockquote>
<p>This doesn't need to go in tomorrow's 2.3.4 release. Let's put this in 2.3.5 or later.</p> CUNY Academic Commons - Bug #19436 (Resolved): Doc nav displays twice on a group doc pagehttps://redmine.gc.cuny.edu/issues/194362023-12-19T03:00:06ZRaymond Hoh
<p>Came across a bug while looking into <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: Audit usage of custom en_CAC locale (Resolved)" href="https://redmine.gc.cuny.edu/issues/19415">#19415</a> where the doc nav was displaying directly below the main navigation, but before the main group interface. Here's an example doc on production with this problem: <a class="external" href="https://commons.gc.cuny.edu/groups/bikecuny/docs/bike-shops/">https://commons.gc.cuny.edu/groups/bikecuny/docs/bike-shops/</a> .</p>
<p>Bug is due to using <code>bp_buffer_template_part()</code> without setting the <code>$echo</code> parameter to <code>false</code> in BuddyPress Docs: <a class="external" href="https://github.com/buddypress/buddypress/blob/39bba862906314d48eeea11ffc127fc8b8fff8c0/src/bp-core/bp-core-template-loader.php#L401">https://github.com/buddypress/buddypress/blob/39bba862906314d48eeea11ffc127fc8b8fff8c0/src/bp-core/bp-core-template-loader.php#L401</a></p>
<p>I've opened a PR for BP Docs here: <a class="external" href="https://github.com/boonebgorges/buddypress-docs/pull/740">https://github.com/boonebgorges/buddypress-docs/pull/740</a>.</p>
<p>Boone, can you review when you have the chance?</p> CUNY Academic Commons - Bug #19204 (Resolved): Github 'cac' repo access rightshttps://redmine.gc.cuny.edu/issues/192042023-11-03T18:51:46ZRaymond Hoh
<p>I'm about to push a fix for <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: New AI Assistant? (Resolved)" href="https://redmine.gc.cuny.edu/issues/19192">#19192</a>, but it seems like my access to <code>cuny-academic-commons/cac/</code> Github repo has been removed.</p>
<p>Boone, can you check my permission access to the <code>cac</code> Github repo?</p> CUNY Academic Commons - System Upgrade #19144 (Resolved): Set 'bp-nelo' as template in developer ...https://redmine.gc.cuny.edu/issues/191442023-10-30T22:37:07ZRaymond Hoh
<p>This is just a message for Boone and Jeremy.</p>
<p>If you are running the latest version of <code>2.2.x</code> branch in your development environment, please make sure to set your <code>'template'</code> option to <code>'bp-nelo'</code>. Can do so with WP-CLI:</p>
<pre>
wp option update template bp-nelo
</pre>
<p>Previously, <code>'bp-nelo'</code> was a child theme of the <code>'bp-default'</code> theme, but in <code>2.2.x</code>, bp-nelo is now its own theme, so the template is now <code>'bp-nelo'</code>. I previously enforced the <code>'template'</code> option to <code>'bp-nelo'</code> with a filter, but that filter is now removed as of <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/2582798911a1e990c07fe7194132438c08b696ee">https://github.com/cuny-academic-commons/cac/commit/2582798911a1e990c07fe7194132438c08b696ee</a> .</p>
<p>So this is a just a FYI.</p> CUNY Academic Commons - Bug #18496 (New): PHP 8+ Compatibilityhttps://redmine.gc.cuny.edu/issues/184962023-07-21T22:45:40ZRaymond Hoh
<p>We're currently running PHP 7.4 in our PHPCS set up and on the GC server, but some of us are using PHP 8+ in our development environments. (For example, <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: bp-email-type doesn't properly deal with 'to' arrays (Resolved)" href="https://redmine.gc.cuny.edu/issues/16948">#16948</a>.)</p>
<p>This will be a catch-all ticket to address deprecated notices, fatal errors and the like.</p>
<p>Also, just to note that PHP 7.4 reached end-of-life back in November 2022. Upgrading to PHP 8.2 is probably good to think about when we decide to move hosts. PHP 8.2 is supported until December 2025, while PHP 8.0 will reach end-of-life in November 2023.</p> CUNY Academic Commons - Support #18128 (Resolved): Question about Jetpack on news.commons.gc.cuny...https://redmine.gc.cuny.edu/issues/181282023-04-27T17:04:08ZRaymond Hoh
<p>I was looking at our Cavalcade logs for failed scheduled tasks and most of the recent, failed tasks come from the Jetpack plugin on <code>news.commons.gc.cuny.edu</code>. In an attempt to fix this, I have reset all Jetpack options on <code>news.commons.gc.cuny.edu</code>.</p>
<p>I'm adding Scott and Colin as watchers to this ticket. Do we use Jetpack extensively on <code>news.commons.gc.cuny.edu</code>? If so, can either one of you re-enable and reconfigure the Jetpack modules that were being used on the site? Here's a direct link to the Jetpack modules page: <a class="external" href="https://news.commons.gc.cuny.edu/wp-admin/admin.php?page=jetpack_modules">https://news.commons.gc.cuny.edu/wp-admin/admin.php?page=jetpack_modules</a> .</p>
<p>If Jetpack is not vital for <code>news.commons.gc.cuny.edu</code>, it maybe worthwhile to disable the Jetpack plugin entirely.</p> CUNY Academic Commons - Bug #17651 (Resolved): Remove ability to create Docs for Usershttps://redmine.gc.cuny.edu/issues/176512023-02-10T18:48:01ZRaymond Hoh
<p>Luke stumbled upon this issue on the call today.</p>
<p>If you are logged in and hover over the admin bar sticky menu on the top-right corner, the "My Docs > Create New Doc" menu item is available. If clicked, this allows users to create new docs, however we only want to allow doc creation through an individual group.</p>
<p>Let's remove this menu item.</p> CUNY Academic Commons - Bug #17578 (Resolved): Deactivate Advanced Post Cache pluginhttps://redmine.gc.cuny.edu/issues/175782023-01-27T21:19:37ZRaymond Hoh
<p>According to <a href="https://github.com/Automattic/advanced-post-cache/issues/28" class="external">this Github issue</a>, WordPress 6.1 already has some improvements to deal with caching post IDs that might make the Advanced Post Cache plugin redundant.</p>
<p>The Advanced Post Cache plugin has some oddities that we've run up against with <a href="https://redmine.gc.cuny.edu/issues/13341#note-11" class="external">bbPress</a>, our <a href="https://redmine.gc.cuny.edu/issues/14457#note-21" class="external">cac_course</a> post type and most recently, <a href="https://redmine.gc.cuny.edu/issues/17551#note-2" class="external">Event Organiser</a>. So it's an opportune time to look at retiring it.</p> CUNY Academic Commons - Bug #16351 (Resolved): No new Redmine emails since Sunday?https://redmine.gc.cuny.edu/issues/163512022-07-12T04:27:42ZRaymond Hoh
<p>Has anyone received any new emails from Redmine since the new Redmine update was deployed during the maintenance window on Sunday?</p>
<p>I just manually checked Redmine to find some new comments, but I did not receive the emails to those comments like I usually do.</p> CUNY Academic Commons - Bug #16176 (Resolved): Set up app password for GMail IMAP accounthttps://redmine.gc.cuny.edu/issues/161762022-05-30T19:27:53ZRaymond Hoh
<p>GMail is pushing for OAuth authentication for less secure, 3rd-party apps by the end of month. So for apps relying on the GMail password like our GMail IMAP account, we will need to set up 2FA and then set up an App Password for use solely with RBE.</p>
<p>Instructions can be found here:<br /><a class="external" href="https://support.google.com/accounts/answer/185833?hl=en#app-passwords">https://support.google.com/accounts/answer/185833?hl=en#app-passwords</a></p>
<p>Boone, I believe the original <code>cunyacademiccommons</code> GMail address was set up by you. Can you set up 2FA and then set up an app password for use with the Commons? Then, can you pass on the app password to me? If you're going to use TOTP for the 2FA, perhaps you can also pass the TOTP secret key to me. This isn't necessary, but will help if another person needs to access the GMail account. If you're using Android, <a href="https://getaegis.app/" class="external">Aegis Authenticator</a> works well, but similar apps would also work.</p>
<p>We should probably transition away from IMAP mode for RBE and switch to Inbound mode. I'll create another issue just for this.</p> CUNY Academic Commons - Bug #14908 (Resolved): Stale object cache on cdevhttps://redmine.gc.cuny.edu/issues/149082021-10-27T20:19:24ZRaymond Hoh
<p>This is a note for Boone that I've experienced some weird instances on cdev with stale object cache.</p>
<p>I noticed this in the admin area yesterday because the BuddyPress update routine kept running on each page load. The value for <code>bp_get_option( '_bp_db_version' )</code> kept returning the version before BuddyPress v8.0 (12385), whereas the version in the DB was the latest one (12850). <a href="https://github.com/buddypress/buddypress/blob/3922b761777c194fa0b05898b9dd636a5928d09b/src/bp-core/bp-core-update.php#L623" class="external">As a result</a>, the BP emails kept on installing on each admin page load. See <a class="external" href="https://commons.gc.cuny.edu/wp-admin/edit.php?post_status=trash&post_type=bp-email">https://commons.gc.cuny.edu/wp-admin/edit.php?post_status=trash&post_type=bp-email</a> where I've cleaned up all instances this has occurred.</p>
<p>Deleting the options object cache fixed this -- <code>wp_cache_delete( 'alloptions', 'options' )</code>.</p>
<p>This also fixed an issue just now where the homepage was throwing a fatal error because the CAC Home Creation plugin was not activated because the object cache was referencing the older '<code>active_plugins</code>' option.</p>
<p>Has something changed to our memcached set up that might explain the stale object cache?</p> CUNY Academic Commons - Bug #12551 (Resolved): Preparing for PHP 7.4https://redmine.gc.cuny.edu/issues/125512020-03-16T22:28:26ZRaymond Hoh
<p>Previously: <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: PHP 7.2 Incompatibility Problems (Resolved)" href="https://redmine.gc.cuny.edu/issues/10564">#10564</a> for PHP 7.2.</p>
<p>This is a master ticket to gather fixes related to PHP 7.4, which will be applied to the production server in a few months from now.</p> CUNY Academic Commons - Bug #11297 (Resolved): Two versions of the same plugin and theme existhttps://redmine.gc.cuny.edu/issues/112972019-04-08T18:02:48ZRaymond Hoh
<p>In Git, you can have folder names with different capitalization and they will be distinct.</p>
<p>For example, <code>wp-content/themes/LayerSlider</code>:<br /><a class="external" href="https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/plugins/LayerSlider">https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/plugins/LayerSlider</a></p>
<p>Is different than <code>wp-content/themes/layerslider</code>:<br /><a class="external" href="https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/plugins/layerslider">https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/plugins/layerslider</a></p>
<p>We need to remove the duplicate ones:<br />- In <code>/wp-content/plugins/</code>, <code>LayerSlider</code> should probably be kept, while the other one should be removed.<br />- In <code>/wp-content/themes/</code>, there are two <code>Basic</code> themes, but they are both different...:<br /><a class="external" href="https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/themes/Basic">https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/themes/Basic</a><br /><a class="external" href="https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/themes/basic">https://github.com/cuny-academic-commons/cac/tree/1.14.x/wp-content/themes/basic</a></p>
<p>I would make the LayerSlider change, but Git on Windows is a little problematic:<br /><a class="external" href="https://docs.microsoft.com/en-us/azure/devops/repos/git/case-sensitivity">https://docs.microsoft.com/en-us/azure/devops/repos/git/case-sensitivity</a></p>
<p>Not sure what to do with the <code>Basic</code> themes. Maybe nothing needs to be done.</p>
<p>Boone, I'm assigning this to you since I do not develop on a Linux OS (although I really should!).</p> CUNY Academic Commons - Bug #9639 (Resolved): Users having problems posting to the forumshttps://redmine.gc.cuny.edu/issues/96392018-04-19T20:12:55ZRaymond Hoh
<p>Received a report that users are having problems posting on the cbox.org forums.</p>
<p>I can verify the report when logging in with an account with the role of <code>subscriber</code>. However, I can post with my admin account.</p>
<p>When attempting to add a forum post as a subscriber, the post submits to the group homepage and doesn't redirect to the forum topic. The post isn't in the "Replies > Spam" admin dashboard page either.</p>
<p>I don't have access to debug the issue further.</p>
<p>Boone, do we have any custom code on cbox.org that might be causing this issue?</p> CUNY Academic Commons - Bug #7825 (Rejected): WP Cron fails to spawn on productionhttps://redmine.gc.cuny.edu/issues/78252017-03-22T17:31:27ZRaymond Hoh
<p>I was checking to see if a scheduled task that I added to WP cron was there or not and it appears that WP cron doesn't appear to be working.</p>
<p>I used the following wp-cli command to see if WP cron was functioning correctly:<br /><code>wp cron test</code></p>
<p>And that throws back the following error:</p>
<p><code>Error: WP-Cron spawn failed with error: cURL error 28: Operation timed out after 3001 milliseconds with 0 out<br />of -1 bytes received</code></p>
<p>I then checked the cron schedule with:</p>
<p><code>wp cron event list</code></p>
<p>And there are a bunch of older cron events dating as far back as March 7th that haven't run. This might be the cause of other issues such as #7346. Also, I'm guessing that no one has received a daily digest since March 7th.</p>
<p>I did some brief research to see what might be causing the problem and a Github user notes that <code>commons.gc.cuny.edu</code> might not be in the HOSTS file during the cURL lookup:<br /><a class="external" href="https://github.com/Automattic/vip-quickstart/issues/421#issuecomment-91355870">https://github.com/Automattic/vip-quickstart/issues/421#issuecomment-91355870</a></p>
<p>We'd have to confirm with Lihua if that is the case or not.</p>
<p>I also just tested WP cron on the development server and it appears to be working there though.</p>