https://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2012-08-14T03:09:12ZCUNY Graduate Center - Project Tracking SystemCUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=84702012-08-14T03:09:12ZBoone Gorgesboone@gorg.es
<ul><li><strong>Target version</strong> changed from <i>1.4.2</i> to <i>1.4.3</i></li></ul> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=85582012-08-20T21:13:38ZMatt Goldmattgold@gmail.com
<ul><li><strong>Target version</strong> changed from <i>1.4.3</i> to <i>1.4.7</i></li></ul> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=92182012-10-01T21:12:01ZBoone Gorgesboone@gorg.es
<ul><li><strong>Assignee</strong> changed from <i>Boone Gorges</i> to <i>Dominic Giglio</i></li></ul><p>This is probably a theme issue. Matt, can you please share a link to the post in question?</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=92712012-10-03T23:57:14ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Yes, Matt, could you please provide a link as Boone requested? I know I've seen Brian's weekly roundups in the past but can't seem to find the site he posts them on. Thanks.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=92752012-10-04T05:43:46ZMatt Goldmattgold@gmail.com
<ul></ul><p>It's the <a href="http://bfoote.commons.gc.cuny.edu/" class="external">Footenotes blog</a></p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93072012-10-04T21:38:53ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>I agree with Boone, this appears to be a theme related issue.</p>
<p>The Footenotes (great name by the way) blog is running the Simplex theme. The comments template file doesn't do a whole lot when it comes to the comments section. It basically makes a call to WP's internal <code>comment_form()</code> function. Contrast this with the Ground Control blog, which uses the Spectrum theme from WooThemes; that theme defines a few related files and functions to customize the way comments and the comment form are displayed, like making decisions about who's logged in or not.</p>
<p>I don't think I'd categorize this as an issue per se. This is more along the lines of a lack of functionality on the part of the theme. However, I'm not marking this as resolved just yet.</p>
<p>Boone, let me know if you agree. I may be missing something in the plain vanilla wp comment form function. Maybe there really is a deeper underlying issue here?</p>
<p>Matt, is the requirement to enter user info when commenting on Footenotes a problem? Maybe we should have Brian look at alternate themes (if Boone doesn't disagree with my assessment that is)?</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93142012-10-05T08:37:23ZBoone Gorgesboone@gorg.es
<ul></ul><p>Thanks for the explanation, Dom. My understanding of the <code>comment_form()</code> function is that it <strong>should</strong> be context-specific for logged-in users. Would you mind looking a little deeper to see if I'm off base here? wp-includes/comment-template.php, starting around 1550.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93442012-10-07T20:40:49ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>OK, first of all, Boone, you were absolutely correct. WP 3.0 introduced the <code>comment_form()</code> function which standardizes the display and control/customization of a site's comment form. It adds a slew of actions and filters that can be used to control a comment form. So the issue is not in WP's commenting subsystem.</p>
<p>I don't know how I got it into my head that the Footenotes blog was using the Simplex theme, but it's not. It's using Manifest. I went through theme after theme, the Codex, and countless web searches to find a solution to this. On the bright side, I learned more than I wanted to know about WP commenting! :-)</p>
<p>Here's the problem. The Manifest theme author never updated the commenting to support 3.0+ features. It's <code>comments.php</code> template still hard-codes a comment form on posts. Therefore, the status of logged in users <strong>IS NOT</strong> used when displaying the form. The only solution to this issue would be to have the author update the theme or for Brian to pick a new theme.</p>
<p>Here's the template in question: <a class="external" href="https://github.com/castiron/cac/blob/master/wp-content/themes/manifest/comments.php">https://github.com/castiron/cac/blob/master/wp-content/themes/manifest/comments.php</a></p>
<p>The pre-3.0 <code><form></code> code starts on line 70.</p>
<p>Matt, I'm sorry but I spent way more time on this issue than I probably should have. When I come across a situation where I can't figure something out, I am unable to concentrate on anything else until I've resolved it.</p>
<p>I'm leaving this issue open for now, since 1.4.7 isn't slated for release until Thurs. Let me know how you'd like me to proceed. I feel like this can be marked as resolved or even rejected as this is not a bug in The Commons but a failing of a third party theme.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93462012-10-07T23:39:23ZMatt Goldmattgold@gmail.com
<ul></ul><blockquote>
<p>Matt, I'm sorry but I spent way more time on this issue than I probably should have. When I come across a situation where I can't figure something out, I am unable to concentrate on anything else until I've resolved it.</p>
</blockquote>
<p>I know it's a process and that you learn lots of things along the way that increase your WP knowledge and may help with future tickets. And that tenacity is what makes you a good programmer. Thanks for your work on this.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93512012-10-08T12:35:33ZBoone Gorgesboone@gorg.es
<ul></ul><p>Thanks for your research, Dom.</p>
<p>I don't want us to be in the business of bringing obsolete themes up to date. So please don't go refactoring <code>comments.php</code> in that theme. However, it does look like the Manifest team is still being maintained, in this case by the (excellent) wordpress.com theme team: <a class="external" href="https://wpcom-themes.svn.automattic.com/manifest/">https://wpcom-themes.svn.automattic.com/manifest/</a> Dom, could I ask you to do the following: On your local, set up a simple site using our existing Manifest theme. Then replace our files with an svn checkout of the wpcom-themes version. Check your local test site to make sure that there are no major changes in the way the theme looks. In short, make sure that the wpcom-themes version is an <strong>update</strong>, rather than a total fork, of the original Manifest.</p>
<p>If the theme looks similar enough, let's just replace our outdated version and call it a fix. Otherwise, I'll mark this ticket as Rejected and we can revisit when we do a more general theme review.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93562012-10-08T21:25:52ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/802">Screen_Shot_2012-10-08_at_5.19.36_PM.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/802/Screen_Shot_2012-10-08_at_5.19.36_PM.png">Screen_Shot_2012-10-08_at_5.19.36_PM.png</a> added</li></ul><p>Boone,</p>
<p>I did as you asked and don't see any major differences. I kept switching back and forth between activating our repo's version and the svn checkout and they look <em>almost</em> identical. There is only one issue I want to get an answer to: The comment form (which is why we're doing this) looks a little "broken?" Broken isn't really the right word, but I'm having trouble coming up with another (better) adjective. I've attached a screen shot of what the comment form looks like when the <strong>svn checkout</strong> of Manifest is active.</p>
<p>Let me know what you think. I think it's just a small styling issue. I don't mind tweaking it if it means Brian will have an updated version for Footenotes.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93602012-10-08T22:50:53ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/803">Screen_Shot_2012-10-08_at_6.49.27_PM.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/803/Screen_Shot_2012-10-08_at_6.49.27_PM.png">Screen_Shot_2012-10-08_at_6.49.27_PM.png</a> added</li></ul><p>Here's a another screenshot of the comment form when you're logged out.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93612012-10-08T23:15:28ZBoone Gorgesboone@gorg.es
<ul></ul><p>Hi Dom. I think the comment form looks fine. Don't spend any time fixing it (it'd just get overwritten in future upgrades). Go ahead and commit the swapped out version.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=93622012-10-09T01:54:00ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Updated manifest theme has been uploaded.</p>
<p>Commit: <a class="external" href="https://github.com/castiron/cac/commit/81346db9f039b7a3efaef7e1c690e928054911eb">https://github.com/castiron/cac/commit/81346db9f039b7a3efaef7e1c690e928054911eb</a></p>
<p>I've also added a note to <a href="https://github.com/castiron/cac/wiki/Release-ACTION_REQUIRED-list" class="external">ACTION_REQUIRED</a> to remind Brian to double check Footenotes after the 1.4.7 release.</p> CUNY Academic Commons - Bug #2044: Comment field asks for name info when user logged inhttps://redmine.gc.cuny.edu/issues/2044?journal_id=94442012-10-12T00:34:58ZBoone Gorgesboone@gorg.es
<ul></ul><p>Hi Brian - Read the above thread only if you want :) Long story short, the theme you use on Footenotes was waaay out of date. In 1.4.7 we brought it up to date, but it's possible that you'll see some small changes. If you see anything that needs to be fixed, please don't hesitate to open a ticket.</p>