Massimiliano Arione

Wrong return types in Calendar

1

Calendar class uses `PeriodInterface` as return type for `getYear`, `getMonth`, `getWeek`. Such method should return, respectively, `Year`, `Month`, `Week`, that are implementing `\Iterator`. With the current code, a call to...

getEntityChangeset not working with ODM

4

### Bug Report | Q | A |---------------- | ------ | BC Break | no | Bundle version | 1.19 #### Summary Method `getEntityChangeset` introduced in dbfe95c7cbe04def86b2e64a5c5ff6c9bb9f515d exists only for...

I tried to update our composer.json to use `"doctrine/mongodb-odm": "^2.0"` (instead of currently 1.2), but tests are failing: ``` There were 10 errors: 1) Vich\UploaderBundle\Tests\Command\MappingDebugClassCommandTest::testNotUploadableClass ParseError: syntax error, unexpected '?',...

In `fields.html.twig` (for `vich_image_widget`), we're currently linking the image to `image_uri`. IMHO, if `download_uri` is present, it should be lilnked to it instead. The current situation is misleading, because we're...

Value objects should be serializable

2

I know that classes here are not intended specifically to be value objects, but can be usable as such. So, it would be nice if that classes would be serializable

Add "inversedBy" attribute in annotation. Without it, Doctrine complains: "The field Tag#tagging is on the inverse side of a bi-directional relationship, but the specified mappedBy association on the target-entity Tagging#tag...

npm is not updated

4

I see we're at version 2.1.33 but [npm](https://www.npmjs.com/package/symfony-collection) is still at 2.1.30 (2 years ago)

add alias to subquery in doCalcRowCount()

5

Without such alias, I get this error: "SQLSTATE[42000]: Syntax error or access violation: 1248 Every derived table must have its own alias"

How to "fake" mobile in tests?

2

I need to test a portion of code where `isMobile()` method is used, so I need to get two different cases, `true` and `false`. How can I do it? TIA

This is the current icon:  See https://www.activestate.com/komodo-ide