Duplicate view copies the old view name and ignores the new one

Good observation!

Do you want to try to fix it? core/modules/views_ui/src/ViewDuplicateForm.php would be the URL you need.

I will provide a patch

Lets try

Tested the patch. It fixes the problem.

So we must be missing test coverage of this.

That's what I thought :) I will provide one later today or tomorrow.

Ok, here's the patch. This is my very first Test class so, please, review it carefully.

1. +++ b/core/modules/views_ui/src/Tests/CloneTest.php
   @@ -0,0 +1,41 @@
   +/**
   + * @file
   + * Contains \Drupal\views_ui\Tests\CloneTest.
   + */
   +
   ...
   + * Tests view clone tool.
   + *
   + * @group views_ui
   + */
   +class CloneTest extends UITestBase {
   ...
   +  public function testCloneView() {
   +
   

   It would be great if we could name things here as duplicate, given that also the UI is talking about duplicating.

2. +++ b/core/modules/views_ui/src/ViewDuplicateForm.php
   @@ -73,6 +73,7 @@ protected function actions(array $form, FormStateInterface $form_state) {
   +    $this->entity->set('label', $form_state->getValue('label'));
   

   +1

Thanks for review. I've changed name convention from clone to duplicate.

+1 from me. Patch applies cleanly, all tests are green and what most important - it fixes reported bug :).

From technical side - change is quite simple - nothing suspicious here. Patch includes also new test case for test suite.

Cool, thank you!

Committed 5a3027c and pushed to 8.0.x. Thanks!