block.html.twig variable docs are incorrect

Ah. So it looks like we need to change the docs header in block.html.twig, where it says:
- module: ...
to
- provider: ...
(under the "configuration" item in available variables section).

Good novice issue.

And by the way there are unfortunately 3 block.html.twig files that need this update :(

Not sure, But might be changed in following all places. suggest!

core/modules/block/templates/block.html.twig
core/modules/system/templates/block--system-menu-block.html.twig
core/themes/bartik/templates/block.html.twig
core/themes/classy/templates/block/block--search-form-block.html.twig
core/themes/classy/templates/block/block--system-menu-block.html.twig
core/themes/classy/templates/block/block.html.twig

changed from 'module' to 'provider', do we need to change description too ?

RE #5, this is now a Documentation bug. Please look elsewhere for programming support. Thanks! There are several support options listed if you click on "Support" at the top of Drupal.org, which will take you to:
http://drupal.org/support
There you can find out about the Drupal IRC channels, and the Forums, which are our two main support mechanisms in the Drupal community. You might also try http://drupal.stackexchange.com/

Good luck with your question!

----

So that aside, the patch looks good to me... though you're right, maybe we should change the description. A block can (theoretically) be provided by a module, installation profile, theme, or "core". So we should probably say something about that. Maybe just "module or other provider" ?

Updated

- *   - provider: The module that provided this block plugin.
+ *   - provider: The module or other provider that provided this block plugin.

Thanks! Lastest patch looks fine. Test failure was due to a bunch of tests getting aborted by ... who knows what/why/whom.

+1, thanks!

Hrm, this made me realize that the cache line just below each of the modified lines is also completely obsolete: we need to remove those lines too.

IMO it makes sense to do that here?

Sure, so it sounds like we just need to remove those parts? Unassigning, @krknth if you have time to pick this up feel free to reassign.

Yep, just remove those parts.

Removed Cache parts.

Changing status

@anil280988 : Thanks!

+++ b/core/themes/classy/templates/block/block.html.twig
@@ -9,7 +9,6 @@
  *   - cache: The cache settings.

You forgot this one.

I'm working on

@anil280988 : your removed module, but not added provider ? any reason ?

diff --git a/core/themes/classy/templates/block/block.html.twig b/core/themes/classy/templates/block/block.html.twig
index 8b584b9..6cc472b 100644
--- a/core/themes/classy/templates/block/block.html.twig
+++ b/core/themes/classy/templates/block/block.html.twig
@@ -9,7 +9,6 @@
  * - configuration: A list of the block's configuration values.
  *   - label: The configured label for the block.
  *   - label_display: The display settings for the label.
- *   - module: The module that provided this block plugin.
  *   - cache: The cache settings.
  *   - Block plugin specific settings will also be stored here.
  * - content: The content of this block.

Added patch & interdiff for #7 - #21

#21 is identical to #7, which was RTBC'd in #10, but now with the additional deletions I requested in #12. Which makes this RTBC again :)

Thanks!

Committed d0b0b4a and pushed to 8.0.x. Thanks!

Testbot's lagging.