https://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2012-11-06T00:58:17ZCUNY Graduate Center - Project Tracking SystemCUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=97482012-11-06T00:58:17ZBoone Gorgesboone@gorg.es
<ul></ul><p>Dom - FYI, it looks like the culprit is probably the upgraded Thematic parent theme. If there is a simple fix that we can implement, please do so. If it looks like it's going to be complicated, let's just roll back to the previous version of Thematic for 1.4.10.</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=97772012-11-09T20:27:44ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>Target version</strong> changed from <i>1.4.10</i> to <i>1.4.11</i></li></ul><p>Boone,</p>
<p>You're right, Themeatic made changes to how they internally handle their <code>themeatic_body_class</code> and <code>themeatic_post_class</code> filters. This was causing weird output to show up in the <code><body></code> tag and therefore breaking the MESTC layout.</p>
<p>I've made some changes to MESTC's functions.php that I believe will fix this issue. I used the recommendations from this forum post:</p>
<p><a class="external" href="http://thematictheme.com/forums/topic/upgrading-to-v-1-0-3-1-and-body_class-in-functions-php/page/2/#post-2488">http://thematictheme.com/forums/topic/upgrading-to-v-1-0-3-1-and-body_class-in-functions-php/page/2/#post-2488</a></p>
<p>I also updated Themeatic in our repo to v1.0.3.3-beta available <a href="https://github.com/ThematicTheme/Thematic/archive/1.0.3.3-beta.zip" class="external">here</a>. One of the changes in this version addresses this issue specifically. I will upgrade our repo to the final version once it's released.</p>
<p>It's difficult for me to say for sure that these changes will fix the issue because my local MESTC site doesn't look like the live site. Hopefully when 1.4.10 is released on Sunday the site will be back to normal. I'm moving this issue to 1.4.11 for now, if all goes well I will move the issue back to 1.4.10 and mark it resolved as part of that milestone on Sunday/Monday. I've also added a reminder to <a href="https://github.com/castiron/cac/wiki/Release-ACTION_REQUIRED-list" class="external">ACTION_REQUIRED</a>.</p>
<p>I will email my edited copy of function.php to Kimon so he has an up-to-date version for his own copy of the theme.</p>
<p>Here are my changes: <a class="external" href="https://github.com/castiron/cac/commit/63de971a4f4f4cc48a84c4d2bd63b70b87b0b9f6">https://github.com/castiron/cac/commit/63de971a4f4f4cc48a84c4d2bd63b70b87b0b9f6</a></p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=97872012-11-09T22:43:03ZMatt Goldmattgold@gmail.com
<ul><li><strong>Priority name</strong> changed from <i>Normal</i> to <i>High</i></li><li><strong>Target version</strong> changed from <i>1.4.11</i> to <i>1.4.10</i></li></ul><p>Upgrading the priority since MESTC has an event coming up on Monday and we should try to get this resolved by then. I suggest that if we find, on the 1.4.10 update, that the changes above don't fix the issue, that we upload the older version of the theme to ensure site consistency while we work out other issues on our dev server. It would be excellent if we could have this solved on the 11th or before.</p>
<p>Dom, Boone: thoughts?</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=97902012-11-09T23:01:07ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>It sounds to me like the smartest thing to do is for me to be sitting in front of my computer Sunday night while Boone is releasing 1.4.10. As soon as he tells me it's live, I'll check The Segal Center. If it's not fixed I can then spend the rest of Sunday night tweaking it so that it's good to go for Monday morning. Boone and I can push successive 1.4.10.1, 1.4.10.2, etc, whatever, until it works. Even if that means rolling back to a previous theme version.</p>
<p>I'm not sure if testing on cdev is gonna be any better than my local testing. It's the settings in the production db that I can't replicate. Things like menus and widgets and plugin configs that control a large portion of the front of the site. I can say that I saw the debug error output in my local env and my changes fixed those errors. So I'm pretty confident that the site will look the way it's supposed to on Sunday night, not 100% though because that's impossible. :-)</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=97932012-11-09T23:04:03ZMatt Goldmattgold@gmail.com
<ul></ul><p>Cool. Thanks for your care and attention to this, Dom.</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=97942012-11-09T23:06:02ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>No prob, whatever it takes - I'll make sure they're lookin good on Monday morning. :-)</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98122012-11-10T22:41:42ZKimon Keramidaskimon@keramidas.com
<ul></ul><p>Dom,</p>
<p>Thanks so much for you help. I tried your functions.php file on my local dev and it didn't fix all of the problems. I was however able to spend a couple of hours on the site and I now believe that I have fixed all of the problems(the upgrade had broken our category color palette and that took some fixing on CSS as well as PHP side) on my Local Dev set up and it should be compatible with the version of Thematic Dom noted in his functions.php version 1.0.3.3. The problem was with the way that the site was handling and constructing the "class" setting for body and post sites. I altered the childtheme override code to make it work with the new Thematic code. I think that they are having a little problem with how this is playing with people's sites and it looks like there are further fixes on the horizon that I will be on the lookout for.</p>
<p>I have shared a new version of the MESTC site theme at the following address (<a class="external" href="https://spideroak.com/browse/share/Kimon/mestcKK12commons">https://spideroak.com/browse/share/Kimon/mestcKK12commons</a>) and that should solve the issues (as well as squash a couple of pre-existing non-critical bugs). Hope that this gets in in time for 1.4.10.</p>
<p>By the way here are the files that I changed:</p>
<p>mestc/functions.php<br />mestc/library/styles/default.css<br />mestc/events/events.css</p>
<p>Thanks for all your help while I've been on daddy-time. I'm glad I was able to get in some time for a full fix, but Dom's would have been a great partial fix and I appreciate the time you guys spent on that as I was unsure I would be able to do this work.</p>
<p>Cheers<br />Kimon</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98132012-11-10T23:54:01ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Kimon,</p>
<p>You're welcome, glad I could get at least a partial fix in for you. I've added your latest code to the repo so it will be included in the 1.4.10 release. The first thing I'm going to check tomorrow night while Boone and I are pushing the release if the MESTC site. So I will be updating here as soon as I have more info. Matt informed me above that you have an event on Monday that the site needs to be ready for. We're gonna do all we can to make sure that happens.</p>
<p>As far as Themeatic goes, from what I read in a few forum posts, this update did indeed break quite a few sites. They deprecated some pretty important functionality. But there are times when this is necessary and unfortunately fixes like these are the only way to get through it. I'll be back with an update as soon as I've seen how the new code affects the site.</p>
<p>Commit: <a class="external" href="https://github.com/castiron/cac/commit/8962f9ddf7d0c863db75434475a99ca32cf05005">https://github.com/castiron/cac/commit/8962f9ddf7d0c863db75434475a99ca32cf05005</a></p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98142012-11-10T23:58:40ZMatt Goldmattgold@gmail.com
<ul></ul><p>Thanks, all, for your work on this. Moving forward, we should always keep theme files updated with new versions, since they could potentially address security vulnerabilities, but now we know that the Thematic developers have a history of breaking sites with their upgrades, so we should pay attention to MESTC the next time we upgrade Thematic.</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98152012-11-11T00:30:55ZKimon Keramidaskimon@keramidas.com
<ul></ul><p>My thoughts exactly Matt. If you guys do know when you are doing a Thematic upgrade in the future let me know so that I can do a test on my local dev setup to preempt any breaking.</p>
<p>Once again thanks for everything and thanks for paying attention to this tomorrow Dom</p>
<p>K</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98382012-11-12T02:08:13ZBoone Gorgesboone@gorg.es
<ul></ul><p>Dom - FYI, I'll be doing this release in the next hour or so. I'll ping you when it's ready, so that you can have a look.</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98572012-11-12T03:32:55ZBoone Gorgesboone@gorg.es
<ul></ul><p>The 1.4.10 release is live. Take a gander.</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=98582012-11-12T03:35:58ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Kimon,</p>
<p>MESTC is lookin' good again. I think our edits did the trick. Please take a look and verify. I'm leaving this issue open until you confirm that everything is working as you expect.</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=99222012-11-14T17:01:30ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Matt,</p>
<p>I'm sure Kimon is still pretty busy with the new baby. Since we haven't heard anything more about problems with MESTC, and their Monday event has passed, do you think I can safely close this issue?</p> CUNY Academic Commons - Bug #2230: MESTC page scrambledhttps://redmine.gc.cuny.edu/issues/2230?journal_id=100652012-11-20T19:39:50ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Resolved</i></li></ul><p>Since we haven't heard back about problems with the MESTC site, I'm marking this issue as resolved.</p>
<p>Please open a new issue if any new errors are experienced.</p>