Closed (fixed)
Project:
Documentation
Component:
Correction/Clarification
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
27 Aug 2010 at 17:23 UTC
Updated:
3 Jan 2014 at 02:04 UTC
Jump to comment: Most recent
Node [70151] on installing contributed modules is for 5.x and 6.x but is not correct for Drupal 7 - this is now done at http://drupal.org/node/895232 this should be reviewed.
Comments
Comment #1
skesslerUpdate on this:
I did not know about the update.module know let's you upload module or theme....eek! So. I went to add this to the documentation http://drupal.org/node/895232 - as you can see it is part of the way done and I found a bug. There is a ticket marked duplicate about this bug http://drupal.org/node/738600 but it links to another thing that I am not sure its related. The issue is that you can't FTP to a server without FTP and my test server fits the bill. I don't know how to document this. I also want to warn people that FTP is not safe but its the only option we give? hmmmm....
I am sure the security of this has been hashed out and rehashed but I am not sure what to do with this.
Thanks,
Steve
Comment #2
arianek commentedGreat work getting this going! Thanks!
I was going to say that this page probably shouldn't be split off for D7 (as that's sort of the mantra), but in this case I think I can actually support a separate page, since methods have changed quite significantly for this version.
Is FTP the only method when using the Update module? (Sounds like from the linked issue.)
We can of course (and I would think, we should) have command line instructions as well, for those who may not use Update module...
Comment #3
arianek commentedps. I see there are a few command line notes, but I am thinking a full set of instructions for command-line only. I have made some small updates to the grammar/formatting, and added an image at the end (re: subscribing to issue updates)
Comment #4
leehunter commentedA couple of comments re this page:
Comment #5
arianek commentedtags
Comment #6
arianek commentedTidied up the 5/6 page http://drupal.org/documentation/install/modules-themes/modules-5-6
Really not sure what to do with that FTP issue http://drupal.org/node/738600, might need to follow up with a core dev.'
Still need to review http://drupal.org/documentation/install/modules-themes/modules-7
Comment #7
arianek commentedtags
Comment #8
jn2 commentedI'll be working on it as part of the April 2011 Docs sprint.
From what I see, the changes suggested in #3 and #4, were never incorporated. So to begin with, I'll reorganize it along those lines, then see what it needs from there. Have also changed the status to 'needs work'.
Sprint page is at http://groups.drupal.org/node/138994.
Comment #9
jn2 commentedI've done the initial reorganization, but there are still things left to do.
Comment #10
arianek commentedgreat, thanks for picking this up!
Comment #11
jn2 commentedThis is now ready for review.
Comment #12
Carolyn commentedThis page looks great to me.
I have a few minor suggestions. I think it would be nice if the note about FTP explained that only having SFTP would be a cause, in which case you can use the manual method.
Since these do seem to be coherent steps, should they be ordered lists?
Also, here:
This implies you can get to permissions from the module page without the 'Administer permissions' permission, which isn't the case.
Comment #13
jn2 commented@Carolyn
The page originally used an ordered list, but that had to go. As pointed out in comment #4, the sections on installing through the interface and manually are a choice. You must do one or the other, not both, so they don't follow sequentially.
Good catch on the permissions section. I made the change. Thanks!
Comment #14
Carolyn commentedi meant an ol under each heading
Comment #15
Carolyn commenteddup
Comment #16
jn2 commentedYou could do that, but the lists are so short. You'd have only 2 under the manual installation, and 4 under enable and configure.
Comment #17
arianek commentedhey jn2 - looks like you did move to ol's here - http://drupal.org/node/895232 is looking pretty good. are there other pages that need reviewing otherwise? (thanks for jumping on the reviews here carolyn!)
Comment #18
jn2 commentedHey arianek,
Umm, I'm a little confused. http://drupal.org/node/895232 is all ul's. But the exchange with Carolyn made me realize that some of the formatting was inconsistent, so I added ul's to the first sections as well and cleaned up some terms.
This is the only page I worked on. That's the focus of this issue, so I think this is done.
I'm assuming someone else should mark it fixed.
Comment #19
arianek commenteddurrr. i totally meant ul's. looks great - i'll mark this fixed!
Comment #20
arianek commentedps. I just read this over one final time and made a couple changes (plus rolled comments) http://drupal.org/node/895232/revisions/view/1435934/1440038
If someone can have a quick glance mainly at my comment about version control, and then the info about the option of creating modules subdirectories, that'd be great. (Fix or reopen if there's anything you feel is not right!)