Alejandro Ibarra
Alejandro Ibarra
@gnongsie Hi Gabriel, could you please review and merge this? thank you
@othercorey Hi, so your suggestion would be adding both columns to $_accessible for multi-column primary key?
@othercorey Hi, I was trying to reproduce this bug using a table with a field `theme_id` and no themes table but the validation was generated correctly: ``` $validator ->nonNegativeInteger('theme_id') ->requirePresence('theme_id',...
@ADmad Same situation than #561 psalm reports: ``` Error: vendor/laminas/laminas-diactoros/src/functions/marshal_headers_from_sapi.php:29:18: ParseError: Syntax error, unexpected T_STRING, expecting T_PAAMAYIM_NEKUDOTAYIM on line 29 (see https://psalm.dev/173) Error: vendor/laminas/laminas-diactoros/src/functions/marshal_headers_from_sapi.php:29:28: ParseError: Syntax error, unexpected T_VARIABLE, expecting...
> > If you have any clue about how to fix it, it is more than welcome :) > > > > When I've run into this in the past...
@ravage84 @Neluxx We are going to spend some time implementing this feature but we would love to hear your suggestions about how would you like this to work including the...
@MasterOdin changes requested has been pushed. For some reason coverage has decreased. One of the issues reported is because of splitting the exception message to avoid PHP CS warning but...
> @MasterOdin changes requested has been pushed. For some reason coverage has decreased. One of the issues reported is because of splitting the exception message to avoid PHP CS warning...
> The decrease in code coverage is fine. Most likely some of the formatting changes made here touch upon code that's not tested. Great then, so it is ready to...
@romanzks Resolved in MR #2085 . Pending review and merge