Skip to content

Conversation

@uestla
Copy link
Contributor

@uestla uestla commented Nov 29, 2012

No description provided.

@fprochazka
Copy link
Member

Thanks for report! But this repository is read-only, and is generated from Kdyby/Framework. I cannot accept pull requests here.

@fprochazka fprochazka closed this Nov 29, 2012
@fprochazka
Copy link
Member

Will you please recreate the pull at Kdyby/Framework, or should I?

@uestla
Copy link
Contributor Author

uestla commented Dec 3, 2012

"Oops! I did it again..." - Britney Spears (https://github.com/Kdyby/Framework/pull/51)

jakubboucek referenced this pull request in jakubboucek/KdybyFormsReplicator Apr 18, 2025
Here is incompatibility with `iterator_to_array()` before PHP 8.2
```
TypeError: iterator_to_array(): Argument #1 ($iterator) must be of type Traversable, array given
```
solcik pushed a commit that referenced this pull request Apr 18, 2025
Here is incompatibility with `iterator_to_array()` before PHP 8.2
```
TypeError: iterator_to_array(): Argument #1 ($iterator) must be of type Traversable, array given
```
jtojnar added a commit to jtojnar/FormsReplicator that referenced this pull request Jun 8, 2025
291a615 introduced type hints so the method now expects a string for name, consistent with `nette/component-model` (nette/component-model@47dab8f).

This unfortunately leads to an error when rendering a form since range produces integers:

    Kdyby\Replicator\Container::createOne(): Argument Kdyby#1 ($name) must be of type ?string, int given
jtojnar added a commit to jtojnar/FormsReplicator that referenced this pull request Jun 8, 2025
291a615 introduced type hints so the method now expects a string for name, consistent with `nette/component-model` (nette/component-model@47dab8f).

This unfortunately leads to an error when rendering a form since range produces integers:

    Kdyby\Replicator\Container::createOne(): Argument Kdyby#1 ($name) must be of type ?string, int given
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants