book.export:
  class: Drupal\book\BookExport
  arguments: ['@entity.manager', '@book.manager', '@renderer']

Here is the constructor signature.
https://github.com/drupal/core/blob/8.6.1/modules/book/src/BookExport.ph...

CommentFileSizeAuthor
#4 book_export_service_3003106-4.patch537 bytesdhirendra.mishra

Comments

Chi created an issue. See original summary.

chi’s picture

chi commented
Title: book.export service declares does not need renderer » book.export service does not need renderer
dhirendra.mishra’s picture

dhirendra.mishra commented
Assigned: Unassigned » dhirendra.mishra
Status: Active » Needs work
dhirendra.mishra’s picture

dhirendra.mishra commented
Assigned: dhirendra.mishra » Unassigned
Status: Needs work » Needs review
StatusFileSize
new book_export_service_3003106-4.patch537 bytes

Kindly find the patch for same.Please review and test my patch.Thanks

googletorp’s picture

googletorp commented
Status: Needs review » Reviewed & tested by the community

This was a good find, look good and all is green. RTBC

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 4: book_export_service_3003106-4.patch, failed testing. View results

dhirendra.mishra’s picture

dhirendra.mishra commented
Status: Needs work » Needs review
chi’s picture

chi commented
Status: Needs review » Reviewed & tested by the community

Tests a green again.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed 3c41d0c and pushed to 8.7.x. Thanks!

  • catch committed 3c41d0c on 8.7.x 
    Issue #3003106 by dhirendra.mishra, Chi: book.export service does not...

Status: Fixed » Closed (fixed)

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