This is POC for creating / updating data classes for Forms in Nette applications https://doc.nette.org/en/3.1/form-presenter#toc-mapping-to-classes

I would be happy if you find time to check if I am on right way how to do it or I am completely missing the point. I don't really like the way how I'm finding $form in class and then its fields, but I didn't find any other way yet.

Next steps should be to find $form->onSuccess callback and change the type of second parameter ($values) to newly class created.

I would be also grateful if you give me some hint how should I test this type of rector (where new class is created and old class is also changed)

Fix to avoid error:

1) Rector\Nette\Tests\Rector\Neon\RenameMethodNeonRector\RenameMethodNeonRectorTest::test with data set #0 (Symplify\SmartFileSystem\SmartFileInfo Object (...))
Error: Call to undefined method Nette\Neon\Node\BlockArrayNode::getSubNodes()

• [x] Update to nette/neon ^3.3.3
• [x] Replace $node->getSubNodes() to directly use $node on loop.

Revert https://github.com/rectorphp/rector-nette/pull/57 due to use only increment when different object update on https://github.com/rectorphp/rector-symfony/pull/120

@TomasVotruba this is based on error https://github.com/rectorphp/rector-src/runs/4649685575?check_suite_focus=true#step:14:81 , I tried to apply the changes of return of FIle processor return on LatteFileProcessor and NeonFileProcessor

There is already PropertyFetchAnalyzer->isLocalPropertyFetchName() that verify local property fetch, check against both this and self usage, so moving to PropertyFetchAnalyzer->isLocalPropertyFetchName() instead of $this->nodeNameResolver->isLocalPropertyFetchNamed()

The getObjectType() is deprecated at rector-src's AbstractRector

https://github.com/rectorphp/rector-src/blob/c6473c6a0b4ecaab710353e83f0c740e2e998d26/src/Rector/AbstractRector.php#L335-L338

This PR update to use NodeTypeResolver->getType()

@TomasVotruba it seems on latest CI run, the test indentation seems removed after services new line. This remove the indentation so the CI will be green.

Fix BetterNodeFinder->findParentTypes() not found, need to loop as ScopeNestingComparator->isInBothIfElseBranch pass a single Node in first parameter.

HEAD is failing, and so are any tests in packages relying on the AbstractRectorTestCase class from Rector

https://github.com/rectorphp/rector-nette/commit/0227bb6f43bde75467ee2c14219713784db47afa

        $file = new File($fileInfo, $fileInfo->getContents());
        $this->applicationFileProcessor->run([$file], $configuration);

Based on https://github.com/rectorphp/rector-src/pull/812 as this direct call is deprecated. I tried to follow return array of nodes of https://tomasvotruba.com/blog/how-to-replace-single-node-with-two-nodes-in-abstract-syntax-tree/ and seems failure on assign and method call:

            return [
                new Expression($assign),
                new Expression($node)
            ];

It got error:

Rector\Nette\Tests\Rector\MethodCall\AddNextrasDatePickerToDateControlRector\AddNextrasDatePickerToDateControlRectorTest::test with data set #3 (Symplify\SmartFileSystem\SmartFileInfo Object (...))
LogicException: leaveNode() may only return an array if the parent structure is an array

so use NodesToAddCollector property for now.