Book administration is a site building task more than a content task - particularly since moving individual items around in books can be done from node/add / edit / 'add a child page'. So this patch just moves the item to site building. Didn't fix tests yet - let's see how bad it is.

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

mdupont’s picture

Version: 7.x-dev » 8.x-dev

Bumping to 8.x-dev

catch’s picture

Title: Move book administration to admin/build » Move book administration to admin/structure
avpaderno’s picture

Status: Needs work » Needs review
StatusFileSize
new5.33 KB
klausi’s picture

Status: Needs review » Needs work

admin/build? Doesn't exist anymore. I think we want to move it to admin/structure, as catch indicated by changing the issue title.

avpaderno’s picture

Status: Needs work » Needs review
StatusFileSize
new5.36 KB

How did I misunderstand that?

valthebald’s picture

StatusFileSize
new4.69 KB

Change seems pretty straight forward, rerolling patch to apply to the latest 8.x-dev

valthebald’s picture

Issue tags: +Novice

Marking this as a novice issue

izus’s picture

Issue tags: -Novice

#7: change-book-menu-501472-7.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Novice

The last submitted patch, change-book-menu-501472-7.patch, failed testing.

izus’s picture

Status: Needs work » Needs review
StatusFileSize
new4.58 KB

rerolling #7

Status: Needs review » Needs work

The last submitted patch, change-book-menu-501472-11.patch, failed testing.

izus’s picture

Status: Needs work » Needs review
StatusFileSize
new7.53 KB

oups forgot to modify paths for test file too.
here is a new one

Status: Needs review » Needs work

The last submitted patch, change-book-menu-501472-13.patch, failed testing.

izus’s picture

Status: Needs work » Needs review
StatusFileSize
new9.44 KB

forgot some again!
hope this one will turn green :)

grimreaper’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Title: Move book administration to admin/structure » Change notice: Move book administration to admin/structure
Priority: Normal » Critical
Status: Reviewed & tested by the community » Active

Pinged @catch and he thinks this change is still relevant... so in it goes :)

Committed 501472 and pushed to 8.x. Thanks!

izus’s picture

Added the change notice http://drupal.org/node/1984590

alexpott’s picture

Priority: Critical » Normal
Status: Active » Fixed

Looks good. Thanks!

Tor Arne Thune’s picture

Title: Change notice: Move book administration to admin/structure » Move book administration to admin/structure
David_Rothstein’s picture

Priority: Normal » Critical
Status: Fixed » Needs review
StatusFileSize
new449 bytes

Er, with that patch there's no actual link to it on the Structure page... and I'm pretty sure it would look very odd as a tab there even if it did somehow appear.

Pretty sure you need something like the attached?

alexpott’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs tests

Patch in #21 works for me...

alexpott’s picture

Issue tags: -Needs tests

Urg... removing tag

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Oopsie. :P Alex says thanks from bed. :)

I can't think of a useful way to test this, so ...

Committed and pushed to 8.x. Thanks!

webchick’s picture

Title: Move book administration to admin/structure » Change notice: Move book administration to admin/structure
Status: Fixed » Active
Issue tags: +Needs change record

Actually, this could use a change notice for documentation updates.

alexpott’s picture

Title: Change notice: Move book administration to admin/structure » Move book administration to admin/structure
Priority: Critical » Normal
Status: Active » Fixed
webchick’s picture

Reading! I should try it sometime. :D

Status: Fixed » Closed (fixed)

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

xjm’s picture

Issue tags: -Needs change record

Untagging. Please remove the "Needs change notification" tag when the change notice task is complete.