When a user views the project summary page but is not logged in, it is not indicated to the user that if he was logged in he could create a new issue (support, bug, feature request). This is in contrary to several core modules (forum, comment, etc.) which do make this more obvious to the user.
The attached patch adds a "Login or register" type link in the Support section of the project summary page when the user is not logged in.
This patch depends on a similar patch in project_issue being applied. See that issue at http://drupal.org/node/102395
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | project_login_to_submit_1.txt | 668 bytes | aclight |
| project_login_to_submit.txt | 687 bytes | aclight |
Comments
Comment #1
dwwExtra spaces after the end of the else clause.
Comment #2
aclight commentedI don't understand what you mean? Are you saying there ARE extra spaces or that there SHOULD be extra spaces? I don't see any extra spaces.
Comment #3
aclight commentedOk..I think I understand what you mean by all the whitespace complaints.
Are you talking about this:
Where above I've replaced 4 space chars with 4 * chars?
If that's what you're talking about, those are popping up because my text editor is auto-indent, and so it's indenting for me but then I just leave the line blank.
I'm sure this is editor dependent, but how do other people deal with this? Do they just not use auto indent, or are some editors smart about this and don't actually do the indentation until something is typed on the line?
Or, is there a flag that I can pass to diff that will cause it to drop the spaces?
Sorry for having patches with extra whitespace--I didn't realize it was doing that.
AC
Comment #4
aclight commentedpatch with 4 spaces removed
Comment #5
drewish commentedis theme_project_issue_create_forbidden() defined some place?
Comment #6
aclight commentedsee patch in http://drupal.org/node/102395
Comment #7
dwwhttp://drupal.org/node/102395 needs work, so for now, this is postponed.
Comment #8
aclight commentedNow that http://drupal.org/node/102395 has been committed, I think this is ready to go.
I checked and it still applies (though there is an offset).
Comment #9
dwwReviewed, tested, committed to HEAD, and installed on d.o. Thanks!
Comment #10
(not verified) commented