https://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2015-06-12T01:03:55ZCUNY Graduate Center - Project Tracking SystemCUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=195842015-06-12T01:03:55Zscott vothscott.voth@verizon.net
<ul><li><strong>Subject</strong> changed from <i>Issues after Canvas Update - Help & </i> to <i>Issues after Canvas Update - Help & Support / Codex</i></li><li><strong>Category name</strong> set to <i>WordPress Themes</i></li></ul><p>The Commons Codex and Help & Support each use a child theme of Canvas and are acting very strangely after the upgrade. Sliders not working (on both) and content pushed to the left and navigation displayed weirdly (Codex). I notice the "Commons News" is displaying as expected (and it uses Canvas, but not a child theme). Wondering if the child theme needs some tweaks??</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=195852015-06-12T01:05:44ZMatt Goldmattgold@gmail.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Assigned</i></li><li><strong>Assignee</strong> set to <i>Boone Gorges</i></li></ul><p>Hmmm. And I think that Dom did most of the child theming. Boone, shall we assign to Dan?</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=195882015-06-12T12:30:48ZBoone Gorgesboone@gorg.es
<ul><li><strong>Assignee</strong> changed from <i>Boone Gorges</i> to <i>Daniel Jones</i></li><li><strong>Target version</strong> set to <i>1.8.3</i></li></ul><p>Dan, can you let me know whether you'll be able to look at this soon (within the next day or so)?</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196052015-06-12T21:49:52ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>Sure thing I can look more into this. One thing I noticed recently when working on the issue with the layering on the modal dialogues is that at least on my install the Help & Support site is just using the regular Canvas theme, not the child theme, and that there are very few changes made in the child theme. I think I'm all up to date from Github - am I missing something?</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196062015-06-13T18:01:50Zscott vothscott.voth@verizon.net
<ul></ul><p>Hi Dan - Thanks for looking into this. The only change in the child theme was to create a header that had the CAC logo that links back to the Commons home page. It is being used on the Codex and Help & Support in prod. Maybe in Dev we never switched the sites to use the new child theme?? Not sure.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196092015-06-15T00:29:04ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>I wasn't able to reproduce this locally, but have you tried the fix suggested here: <a class="external" href="https://support.woothemes.com/hc/en-us/articles/203321803-Business-Slider-Images-Not-Showing-Up">https://support.woothemes.com/hc/en-us/articles/203321803-Business-Slider-Images-Not-Showing-Up</a> ?</p>
<p>Right now the slides and everything are working, and all the right HTML is being produced, but the images themselves aren't being requested correctly. It looks like what's happening is for some reason the theme is trying to use the TimThumb script that used to be included with the Woo Framework to get the thumbnail, but in the most recent version of the framework the thumb.php file is empty, so it isn't able to pull anything in. Canvas is supposed some native functions now, but that ends up being dependent on some options that hopefully the fix above can get straight. I noticed that on my local install, the src for the images doesn't include the reference to the TimThumb script, which makes me think this is some kind of issue with the settings, even if it isn't the one above. Let me know if you try that and it still doesn't work and I can keep looking.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196122015-06-15T23:29:36Zscott vothscott.voth@verizon.net
<ul></ul><p>Hi Dan - I tried the fix you suggested - and that did seem to work. Then I started refreshing cache on all browsers (I swear I tried this before :) ) and things are looking much better - navigation is back, sidebar is back - the only thing I notice now that is screwed up is the business slider. I could play around with this myself, or if you want to - that's cool - I don't know how important it is since we were talking about getting rid of it at the last meeting.</p>
<p>Unfortunately, I don't know if our user base will know to clear its cache.</p>
<p>Thanks again for your help!</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196172015-06-16T13:46:24ZBoone Gorgesboone@gorg.es
<ul></ul><p>Thanks for your work on this so far, everyone.</p>
<p>Regarding script/style caches: I've been meaning to do something about this for a while, so this seems like as good an opportunity as any. When WP enqueues a script or style, it uses a query var <code>?ver=4.2.2</code>, where "4.2.2" is the current WP version. When that version number is bumped, the URL of the asset changes, causing the asset to be reloaded from the server (ie, the cache is busted). This gets a bit funky on the Commons, because our release cycle doesn't line up with (and is much more frequent than) WP core releases. I've put a fix into place that makes the 'ver' string look like this: <code>?ver=4.2.2-1.8.3</code>, where "4.2.2" is the WP version and "1.8.3" is the CAC version. This should ensure that browser caches are properly busted when CAC updates.</p>
<p>See <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/1a4f9e9bc313f12b85a0a23a693d232a96a0ad4d">https://github.com/cuny-academic-commons/cac/commit/1a4f9e9bc313f12b85a0a23a693d232a96a0ad4d</a> for the fix.</p>
<p>Note that WP has a disparity between the way script tags and style tags are filtered, which requires a less-than-ideal regular expression in the CAC filter. I've opened a WP ticket to fix the disparity <a class="external" href="https://core.trac.wordpress.org/ticket/32660#ticket">https://core.trac.wordpress.org/ticket/32660#ticket</a> - it's an easy one, so I imagine someone will pick it up for the 4.3 release.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196332015-06-16T22:40:39ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>Thanks for that, Boone!</p>
<p>I'd be happy to play around with this. Just a heads up it looks like a very similar issue has come up with the Simplicity theme on GCDI - <a class="external" href="https://redmine.gc.cuny.edu/issues/4151">https://redmine.gc.cuny.edu/issues/4151</a></p>
<p>I found a quick fix for that for now but I think it probably isn't a coincidence that both are having the same weird issue with generating thumbnails, though I guess it could be.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196372015-06-17T12:51:48ZBoone Gorgesboone@gorg.es
<ul></ul><blockquote>
<p>Just a heads up it looks like a very similar issue has come up with the Simplicity theme on GCDI - <a class="external" href="https://redmine.gc.cuny.edu/issues/4151">https://redmine.gc.cuny.edu/issues/4151</a></p>
</blockquote>
<p>Yeah, this is almost definitely the same problem.</p>
<p>It does seem to have to do with generating thumbnails. There was a problem a few years ago where Timthumb, the library used for on-the-fly image resizing that's packaged with WooThemes, was breaking because it didn't have write permissions to its own directory. So I wrote a script that created a timthumb-config.php file to define a custom cache directory. The related tickets that I see are <a class="issue tracker-1 status-5 priority-5 priority-high4 closed" title="Bug: Vulnerability in timthumb (Resolved)" href="https://redmine.gc.cuny.edu/issues/1099">#1099</a>, <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Support: Image Processing on Byline Theme (Resolved)" href="https://redmine.gc.cuny.edu/issues/1449">#1449</a>, and <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: Homepage Slider Broken on ILETC site: Simplicity Theme (Resolved)" href="https://redmine.gc.cuny.edu/issues/3242">#3242</a>. Dan, I'm not sure if these are related, but it's probably worth taking a look through those tickets and related changesets to see if you get any leads.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=196862015-06-22T13:55:15ZBoone Gorgesboone@gorg.es
<ul><li><strong>Target version</strong> changed from <i>1.8.3</i> to <i>1.8.4</i></li></ul> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=198322015-07-02T14:24:08ZBoone Gorgesboone@gorg.es
<ul><li><strong>Target version</strong> changed from <i>1.8.4</i> to <i>1.8.5</i></li></ul> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=199362015-07-21T14:47:21ZBoone Gorgesboone@gorg.es
<ul><li><strong>Target version</strong> changed from <i>1.8.5</i> to <i>1.8.6</i></li></ul> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=199482015-07-21T21:48:24ZBoone Gorgesboone@gorg.es
<ul></ul><p>It looks like this is the culprit: <a class="external" href="http://www.woothemes.com/2014/12/goodbye-timthumb-thanks-memories/">http://www.woothemes.com/2014/12/goodbye-timthumb-thanks-memories/</a></p>
<p>We had a similar problem at <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: Authentication request (Resolved)" href="https://redmine.gc.cuny.edu/issues/4315">#4315</a>. Running the "Remove TimThumb" tool seems to have fixed everything. Could you please try it and let me know?</p>
<p>I don't see an obvious way to force this routine to be run, in the background, for all relevant sites. So either (a) someone will have to do it manually for all sites running WooThemes themes, or (b) we do nothing, but direct people to run the tool themselves if they run into any problems.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=200462015-07-31T16:08:39ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>Thanks for figuring this out, Boone! Seems to have fixed the same issue locally for me, too.</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=200642015-08-02T12:50:25ZBoone Gorgesboone@gorg.es
<ul><li><strong>Target version</strong> changed from <i>1.8.6</i> to <i>1.8.7</i></li></ul> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=201262015-08-08T22:53:43ZDaniel Jonesdrjones18@gmail.com
<ul></ul><p>Should we close this issue do you think? Did we pick one of the options that Boone laid out?</p> CUNY Academic Commons - Bug #4143: Issues after Canvas Update - Help & Support / Codexhttps://redmine.gc.cuny.edu/issues/4143?journal_id=201382015-08-10T18:44:28ZBoone Gorgesboone@gorg.es
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Resolved</i></li><li><strong>Target version</strong> changed from <i>1.8.7</i> to <i>Not tracked</i></li></ul><p>I think we can close this ticket. Scott, please bookmark this ticket in case you get related support requests in the future.</p>