mbrodala
Repos
197
Followers
31

My Sublime Snippet Collection

0
0

Events

delete branch
mbrodala delete branch patch-1
Created at 2 days ago
issue comment
Broken input with Codeception/WebDriver "fillField()"

Thanks, works great and we could revert our workaround. :+1:

Created at 3 days ago
opened issue
Broken input with Codeception/WebDriver "fillField()"

We have quite a few acceptance tests run by CodeCeption/Webdriver which, among others, fill date picker fields managed by v-calendar. This did work fine using fillField() up until v3.0.0-alpha.8. However, with v3.0.0 this started to fail.

Debugging the test runs which fortunately are recorded as video we saw that filling the field this way leads to broken input: previously the existing date value from the input field was overwritten as desired. Now e.g. entering 23.03.2023 leads to a value like NaN.NaN.0NaN. (We use locale: 'de' for the datepicker, thus the date format is DD.MM.YYYY.)

After some investigation we found that fillField() internally first uses clear() to empty the field and then inputs the desired value via sendKeys(). Evidently after clear() some processing/validation is going on leading to the broken input.

The solution which did work for us was using pressKey() to first select the existing value using Ctrl+A followed by the actual input which then overwrites the previous value:

$I->pressKey(
    // Use selector for date picker field
    ...,
    // Select existing text
    ['ctrl', 'a'],
    // Overwrite with new value
    ...,
);

Not sure if this is something v-calendar can or should fix but we wanted to be sure this issue is mentioned somewhere with a solution to be found by others.

Created at 6 days ago
closed issue
Undefined variable $argv in /typo3fluid/fluid/src/Tools/console.php

The variable $argv does not exist:

msedge_U4iveFxHEp

Created at 1 week ago
issue comment
Undefined variable $argv in /typo3fluid/fluid/src/Tools/console.php

This was reported in https://github.com/TYPO3/Fluid/issues/691 already, please use the search before posting issues. ;-)

Created at 1 week ago
closed issue
Error in 2.7.3 at flow:package:rescan

Version: 2.7.3

When I run the flow flow:package:rescan I get the following error. The error does not occur with version 2.7.2.

Error: Warning: Undefined variable $argv in /data/neos/Packages/Libraries/typo3fluid/fluid/src/Tools/console.php line 22

Created at 1 week ago
Cannot run Composer binaries without path anymore

Expected Behavior

Assuming a Compose service app using this Docker image and phpstan/phpstan is installed via Composer, running its binary vendor/bin/phpstan should work without the path prefix vendor/bin:

± docker compose exec app phpstan
Note: Using configuration file /var/www/html/phpstan.neon.
 453/453 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%


                                                                                                                        
 [OK] No errors                                                                                                         
                                                                                                                        

Current Behavior

Trying to run a Composer binary without path fails:

± docker compose exec app phpstan
OCI runtime exec failed: exec failed: unable to start container process: exec: "phpstan": cannot run executable found relative to current directory: unknown

This seems to be related to Go 1.19 which dropped support for executing commands from the current directory (a "dot path") as a security measurement. Obviously the Docker engine did upgrade to this Go version in the meantime.

This Docker image adds (among others) vendor/bin to the $PATH which did allow for running Composer binaries so far but this has stopped working now:

https://github.com/thecodingmachine/docker-images-php/blob/098c31b33f019fb69f45df85a14f507a4f300fb5/utils/Dockerfile.slim.blueprint#L255L262

Possible Solution

Use absolute paths in the $PATH, so /var/www/html/vendor/bin and /usr/src/app/vendor/bin instead of just vendor/bin depending on CLI/FPM/Apache variant.

Steps to Reproduce (for bugs)

  1. Have a recent Docker engine (23.0 or newer)
  2. Install a Composer package which provides a binary
  3. Use Docker Compose or Docker exec to run that binary just by its name
  4. See the error mentioned above

Context

Developing PHP-based apps. ;-)

Your Environment

  • Version used: 8.1-v4-fpm-node16
  • Operating System and version: Debian Sid
  • Link to your project: n/a
Created at 1 week ago
pull request closed
[TASK] Update phpunit/phpunit (^9.0 => ^10.0)

Mend Renovate

This PR contains the following updates:

| Package | Type | Update | Change | |---|---|---|---| | phpunit/phpunit (source) | require-dev | major | ^9.0 -> ^10.0 |


Release Notes

v10.0.16

Compare Source

v10.0.15

Compare Source

v10.0.14

Compare Source

v10.0.13

Compare Source

v10.0.12

Compare Source

v10.0.11

Compare Source

v10.0.10

Compare Source

v10.0.9

Compare Source

v10.0.8

Compare Source

v10.0.7

Compare Source

v10.0.6

Compare Source

v10.0.5

Compare Source

v10.0.4

Compare Source

v10.0.3

Compare Source

v10.0.2

Compare Source

v10.0.1

Compare Source

v10.0.0

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • [ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

Created at 1 week ago
issue comment
[TASK] Update phpunit/phpunit (^9.0 => ^10.0)

This needs an update of nimut/testing-framework.

Created at 1 week ago
mbrodala delete branch export-ignore
Created at 2 weeks ago

[TASK] Exclude PHPUnit cache from dist archive

[TASK] Exclude PHPStan config from dist archive

[TASK] Exclude vendor directory from dist archive

Merge pull request #36 from pagemachine/export-ignore

Update dist archive exclusions

Created at 2 weeks ago
pull request closed
Update dist archive exclusions
Created at 2 weeks ago
pull request opened
Update dist archive exclusions
Created at 2 weeks ago
mbrodala create branch export-ignore
Created at 2 weeks ago
mbrodala delete branch test-keep-empty-lines-comments
Created at 2 weeks ago

[TASK] Add test for keeping empty lines and comments

Merge pull request #35 from pagemachine/test-keep-empty-lines-comments

Add test for keeping empty lines and comments

Created at 2 weeks ago
pull request closed
Add test for keeping empty lines and comments
Created at 2 weeks ago
pull request opened
Add test for keeping empty lines and comments
Created at 2 weeks ago
mbrodala create branch test-keep-empty-lines-comments
Created at 2 weeks ago
Add fluent interface

I squashed all commits into one and added a missing exception import. Thanks for the contribution!

This was released now: https://github.com/pagemachine/authorized-keys/releases/tag/2.2.0

Created at 2 weeks ago
mbrodala create tag 2.2.0
Created at 2 weeks ago

[FEATURE] Add fluent interface

Merge pull request #34 from vokeit-gschuster/master

Add fluent interface

Created at 2 weeks ago
pull request closed
Add fluent interface

Added fluent interface and fromFile() for PublicKey

Created at 2 weeks ago