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

skessler’s picture

Update 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

arianek’s picture

Great 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...

arianek’s picture

ps. 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)

leehunter’s picture

A couple of comments re this page:

  • In a procedure, the reader would expect that each numbered step is part of a specific sequence that must be followed in order from start to finish. So in this case, step one doesn't really belong (i.e. if you're using the download manager you're not going to move on to step 2 etc.) Also the last step doesn't belong, because it's an "if" statement. That should be moved to the Notes. Actually, there probably should be two sets of bullets at the end, one for "Notes" and the other for "Tips". Tips are more things like subscribing to an issue or asking questions in a forum (i.e. here's something that you can do to make your life easier). An example of a "note" is the bullet about module names (i.e. information that helps you to understand what's happening)
  • There's already a page for Drupal 6 and 7 about using the web interface (http://drupal.org/handbook/modules/update) so we don't need to provide anything more than a link to that page. We shouldn't try to explain and illustrate both approaches in one article.
arianek’s picture

Issue tags: +d7docs

tags

arianek’s picture

Component: New documentation » Correction/Clarification
Status: Active » Needs review

Tidied 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

arianek’s picture

Issue tags: +Sprint: April 2011

tags

jn2’s picture

Assigned: Unassigned » jn2
Status: Needs review » Needs work

I'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.

jn2’s picture

I've done the initial reorganization, but there are still things left to do.

arianek’s picture

great, thanks for picking this up!

jn2’s picture

Status: Needs work » Needs review

This is now ready for review.

Carolyn’s picture

This 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:

Navigate to (Administer > Modules and click on the 'Permissions' button for the desired module. Users with the 'Administer permissions' permission can also do this on the Permissions page (Administer > People > Permissions).

This implies you can get to permissions from the module page without the 'Administer permissions' permission, which isn't the case.

jn2’s picture

@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!

Carolyn’s picture

i meant an ol under each heading

Carolyn’s picture

dup

jn2’s picture

You could do that, but the lists are so short. You'd have only 2 under the manual installation, and 4 under enable and configure.

arianek’s picture

hey 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!)

jn2’s picture

Hey 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.

arianek’s picture

Status: Needs review » Fixed

durrr. i totally meant ul's. looks great - i'll mark this fixed!

arianek’s picture

ps. 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!)

Status: Fixed » Closed (fixed)
Issue tags: -d7docs, -Sprint: April 2011

Automatically closed -- issue fixed for 2 weeks with no activity.