profile
viewpoint
Guilherme Blanco guilhermeblanco Toronto, ON, Canada http://www.doctrine-project.org Core developer of @doctrine, contributor to @php and @symfony.

guilhermeblanco/Doctrine2-Hierarchical-Structural-Behavior 35

Hierarchical Structural Behavior to support Tree structures inside Doctrine2

felipenmoura/theWebMind 26

An innovating way to build your applications from zero, "talking" to your computer

guilhermeblanco/Doctrine2-Sluggable-Functional-Behavior 14

Sluggable Functional Behavior to support slugs inside Doctrine 2

bisna/ksuid 2

KSUID Implementation

guilhermeblanco/create 1

Midgard Create, a generic web editing interface for any CMS

guilhermeblanco/DoctrineDataFixturesExtension 1

Doctrine Data Fixtures Extension for Behat

guilhermeblanco/DoctrineExtensions 1

Doctrine2 behavioral extensions, Translatable, Sluggable, Tree-NestedSet, Timestampable, Loggable, Sortable

guilhermeblanco/fig-standards 1

Standards either proposed or approved by the Framework Interop Group

push eventdoctrine/doctrine-website-build-prod

Doctrine Deployer

commit sha b60d2b44a8452837fe6f7346c4559815fa4100cf

New version of Doctrine website

view details

push time in an hour

issue openeddoctrine/orm

Confusing parameter name $classes

See SchemaTool::getUpdateSchemaSql(array $classes) method.

$classes represents array of ClassMetadata, no string class names.

created time in 2 hours

pull request commentdoctrine/event-manager

Migrate from Travis CI to GitHub Actions

Note: even though this PR does not include PHP 8 and does less than what Travis already did, it still improves greatly the situation since Travis currently refuses to run, moreover the PHP8 job was not marked as required that's why I thought it would be OK to merge.

nicwortel

comment created time in 5 hours

pull request commentdoctrine/event-manager

Migrate from Travis CI to GitHub Actions

Thanks @nicwortel ! Please proceed with the subsequent PRs :)

nicwortel

comment created time in 5 hours

push eventdoctrine/event-manager

Nic Wortel

commit sha 11c983d65291d7fe132f3b9aa37e7b128e1fc493

Migrate from Travis CI to GitHub Actions GitHub Actions configuration is copied from https://github.com/doctrine/lexer/pull/48, except for PHP versions in the build matrix.

view details

Nic Wortel

commit sha 6fa8d3c5ae879d7f171b2ffd4c2450dcaf0909c0

Update dealerdirect/phpcodesniffer-composer-installer The old version did not work with Composer 2.

view details

Nic Wortel

commit sha fc051bd902299b0e7fa999c2edc308d6d4f38e5e

Require PHPStan as a dev-dependency and add phpstan.neon.dist

view details

Nic Wortel

commit sha 500125a41ad8954a2937c5b801f96b29d172e3ae

Fall back to xdebug for code coverage

view details

Nic Wortel

commit sha d120f29ce092373105491545cfe944324c9ce101

Rename step to "Install PHP with Xdebug"

view details

Grégoire Paris

commit sha a894f68b83502ca9a1394307c21180ea65034a97

Merge pull request #34 from nicwortel/github-actions Migrate from Travis CI to GitHub Actions

view details

push time in 5 hours

PR merged doctrine/event-manager

Reviewers
Migrate from Travis CI to GitHub Actions

Travis CI builds are no longer working, so in order to have CI builds again this repository has to be migrated to GitHub Actions.

Also see https://github.com/orgs/doctrine/projects/7#card-50248167, https://github.com/doctrine/migrations/pull/980#issuecomment-736738554.

+262 -72

0 comment

9 changed files

nicwortel

pr closed time in 5 hours

PR opened doctrine/orm

Add psalm annotation to ArrayCollection of Parameters

Without this I always miss Parameters types in collection

+26 -9

0 comment

3 changed files

pr created time in 5 hours

issue closedkrakjoe/apcu

apcu_key_info() not documented

The apcu_key_info() function is currently not documented on php.net.

Implementation is at: https://github.com/krakjoe/apcu/blob/4bbdd30a04f9ca528dda79572ef54dd88b63778b/apc_cache.c#L1077

closed time in 5 hours

nikic

issue commentkrakjoe/apcu

apcu_key_info() not documented

Thanks!

nikic

comment created time in 5 hours

issue commentkrakjoe/apcu

apcu_key_info() not documented

The patch has been submitted to take effect, please close the issue https://www.php.net/manual/zh/function.apcu-key-info.php

nikic

comment created time in 5 hours

pull request commentdoctrine/orm

Make IdentityFunction handle primary keys when they are part of an association

Any news on this?

MatTheCat

comment created time in 8 hours

issue commentdoctrine/orm

Composite foreign keys with shared columns

Hello, sorry to dig up old topics, I'm trying to find a solution as well to the same kind of issue.

I've setup a demo repository where I can experiment. I opened a PR trying to find a solution to this.

Can you have a look and let me know if it is the same kind of issue that we are having?

scaytrase

comment created time in 9 hours

issue commentdoctrine/orm

DDC-3296: JoinColumns seems to only populate one JoinColumn

I just tested with doctrine/orm 2.8.x-dev and this demo repo I cannot reproduce the issue.

I guess this can be closed.

doctrinebot

comment created time in 9 hours

issue commentdoctrine/orm

InExpression will give Syntax error if using non-named empty array

@Ocramius Open for PR on this change as proposed by @mickadoo? It should result in a query like IN(NULL).

mickadoo

comment created time in 10 hours

startedcamptocamp/bivac

started time in 11 hours

startedZettaIO/restic-compose-backup

started time in 11 hours

Pull request review commentphp-fig/fig-standards

Move Beau Simensen to Former Core Committee; Add Vacancy

 Feel free to contact the secretaries at info AT php-fig.org. For more informatio  | Name                                  | Term                    | |---------------------------------------|-------------------------|-| Beau Simensen ([@beausimensen])       | 2016-12-24 - 2020-08-31 |+| Vacant                                |                         | | Larry Garfield ([@Crell])             | 2016-12-24 - 2021-05-30 | | Matthew Weier O'Phinney ([@mwop])     | 2016-12-24 - 2021-05-30 | | Cees-Jan Kiewiet ([@WyriHaximus])     | 2016-12-24 - 2020-08-31 |

@Jean85 When did that re-election happen and what is their term extended to?

SlvrEagle23

comment created time in 13 hours

Pull request review commentphp-fig/fig-standards

Move Beau Simensen to Former Core Committee; Add Vacancy

 Feel free to contact the secretaries at info AT php-fig.org. For more informatio  | Name                                  | Term                    | |---------------------------------------|-------------------------|-| Beau Simensen ([@beausimensen])       | 2016-12-24 - 2020-08-31 |+| Vacant                                |                         | | Larry Garfield ([@Crell])             | 2016-12-24 - 2021-05-30 | | Matthew Weier O'Phinney ([@mwop])     | 2016-12-24 - 2021-05-30 | | Cees-Jan Kiewiet ([@WyriHaximus])     | 2016-12-24 - 2020-08-31 |

We should also update the term of the CC members that got reelected.

SlvrEagle23

comment created time in 13 hours

Pull request review commentdoctrine/event-manager

Migrate from Travis CI to GitHub Actions

+name: "Continuous Integration"++on:+  pull_request:+    branches:+      - "*.x"+      - "master"+  push:+    branches:+      - "*.x"+      - "master"++env:+  fail-fast: true++jobs:+  phpunit:+    name: "PHPUnit"+    runs-on: "ubuntu-20.04"++    strategy:+      matrix:+        php-version:+          - "7.1"+          - "7.2"+          - "7.3"+          - "7.4"++    steps:+      - name: "Checkout"+        uses: "actions/checkout@v2"+        with:+          fetch-depth: 2++      - name: "Install PHP with PCOV"+        uses: "shivammathur/setup-php@v2"+        with:+          php-version: "${{ matrix.php-version }}"+          coverage: "xdebug"

@greg0ire the issue I ran into right now is the PHPUnit version, which I did not want to upgrade in this PR.

nicwortel

comment created time in 15 hours

Pull request review commentdoctrine/event-manager

Migrate from Travis CI to GitHub Actions

+name: "Continuous Integration"++on:+  pull_request:+    branches:+      - "*.x"+      - "master"+  push:+    branches:+      - "*.x"+      - "master"++env:+  fail-fast: true++jobs:+  phpunit:+    name: "PHPUnit"+    runs-on: "ubuntu-20.04"++    strategy:+      matrix:+        php-version:+          - "7.1"+          - "7.2"+          - "7.3"+          - "7.4"

PHP 8 is not tested anymore, and will be the source of a lot more changes I'm afraid… You will have to upgrade doctrine/coding-standard to v8, maybe also PHPStan.

Yeah I already ran into some of those issues when I started working on #33. I have PHP 8.0 installed on my local machine, so at least I can work on this without waiting for CI all the time. I will continue my work when this PR is merged.

nicwortel

comment created time in 15 hours

pull request commentdoctrine/inflector

dutch language

@SenseException

I can't tell anything about the language, but under this premise we should still proceed with this PR even if we can't find someone who can review the parts of the dutch language.

Please also check the failing coding style build.

maybe my previous reply got lost, error on me. this reply: i'd like to keep the comments in as pointers for further work. is okay?

noud

comment created time in 15 hours

startedprofclems/glab

started time in 16 hours

Pull request review commentdoctrine/inflector

dutch language

+<?php++declare(strict_types=1);++namespace Doctrine\Tests\Inflector\Rules\Dutch;++use Doctrine\Inflector\Inflector;+use Doctrine\Inflector\InflectorFactory;+use Doctrine\Inflector\Language;+use Doctrine\Tests\Inflector\Rules\LanguageFunctionalTest;++class DutchFunctionalTest extends LanguageFunctionalTest+{+    /**+     * @return list<array{string, string}>+     */+    public function dataSampleWords(): array+    {+        return [+            ['schip', 'schepen'],+            ['stad', 'steden'],+            ['gelid', 'gelederen'],+            // @todo more+            ['weerman', 'weermannen'],+            ['ei', 'eieren'],+            ['rij', 'rijen'],+            ['mogelijkheid', 'mogelijkheden'],+            ['adres', 'adressen'],+            ['olie', 'oliën'],+            ['industrie', 'industrieën'],+            ['lid', 'leden'],+            ['smid', 'smeden'],+            ['kalf', 'kalveren'],+            ['lam', 'lammeren'],+            ['koe', 'koeien'],+            ['vlo', 'vlooien'],+            ['leerrede', 'leerredenen'],+            ['lende', 'lendenen'],+            ['genius', 'geniën'],+            ['aanbod', 'aanbiedingen'],+            ['dank', 'dankbetuigingen'],+            ['gedrag', 'gedragingen'],+            ['genot', 'genietingen'],+            ['lof', 'lofbetuigingen'],+            ['qaestrices', 'quaestrix'],+            ['matrices', 'matrix'],+            ['twitter', 'twitter'],+            // @todo: multiplitudes array w/o sort order for multiple plural same possibilities+            ['epos', 'epen'],           +            // ['epos', 'epossen'],+            // @todo: multiplitudes array w/o sort order for multiple plural different meenings+            ['beleg', 'belegeringen'],     // @todo: meening: invest a city and then conquer it+            // ['beleg', 'beleggen'],      // @todo: meening: call a meeting+            // ['beleg', 'belegjes'],      // @todo: meening: the slices food on a sandwich

hi, i did put this to resolved basic because i had same dialog w/ @greg0ire me telling i want the comments in because them are pointers to work to do later-on.

noud

comment created time in 17 hours

push eventdoctrine/doctrine-website-build-prod

Doctrine Deployer

commit sha ff984f77a8d648807bcac252dbb075c77d81ecd3

New version of Doctrine website

view details

push time in 18 hours

pull request commentdoctrine/inflector

dutch language

Yes maybe, try composer install?

the github web checks we are talking about.

i just did rm -r vendor && composer install nothing changed so nothing to commit.

i will add one bogus char or line now and we see....

noud

comment created time in 18 hours

PR merged doctrine/lexer

Manually merge 1.2.x into master
+279 -120

0 comment

13 changed files

greg0ire

pr closed time in 18 hours

push eventdoctrine/lexer

Grégoire Paris

commit sha 675b6d742d032e39830992d6d600e0c422869f74

Setup release workflow

view details

Grégoire Paris

commit sha e1eab5e5bdfadd4ce14714988450e571bc1d9f21

Drop commented out code

view details

Grégoire Paris

commit sha 708cefbf56b008af10071524213622fab77d0856

Reuse shared workflows

view details

Grégoire Paris

commit sha 9b87ead646a96d0ea2114c7e2542d39f3cfe978e

Migrate unit tests to Github Actions

view details

Grégoire Paris

commit sha 8667bf0570e5c16d0759c92de5df4606017be23c

Use dependencies compatible with PHP 8

view details

Grégoire Paris

commit sha 969ccf558930a14adb959b1a4f38beda0aff53ea

Upgrade cs lib

view details

Grégoire Paris

commit sha ad32f24e054e9401668e750ca14e596e69af9182

Automated fixes

view details

Grégoire Paris

commit sha 863bff41a710652a73702ee5a18237cb664f160b

Manual fixes

view details

Grégoire Paris

commit sha 20315a7b404c077e4ba10607de735c8c20da4060

Merge pull request #48 from greg0ire/maintenance-work Maintenance work

view details

Grégoire Paris

commit sha cffe3e94d62f717a10a7d8614a19917f9311a5bc

Merge remote-tracking branch 'origin/1.2.x' into master

view details

push time in 18 hours

pull request commentdoctrine/inflector

dutch language

Yes maybe, try composer install?

noud

comment created time in 18 hours

PR opened doctrine/lexer

Manually merge 1.2.x into master
+278 -116

0 comment

12 changed files

pr created time in 18 hours

more