initbiz / money-plugin Goto Github PK

https://github.com/initbizlab/oc-money-plugin/blob/add98559e02bf4be595fa4f3d748b9e3fc887e17/behaviors/MoneyFields.php#L57-L64

When the plugin fetches the current amount stored, it always grabs the 'amount' column. This only works if the actual configured column is also called 'amount' - otherwise the returned current value would be null.

We're running WinterCMS with Laravel 9.1. Laravel uses the ramsey/uuid package which depends on "brick/math": "^0.8 || ^0.9". This plugin depends on "brick/money": "^0.5.3" which in turn depends on "brick/math": "~0.7.3 || ~0.8.0 || ~0.9.0".

It is my understanding that ~0.9.0 does not allow 0.10.x to be installed whereas the ^0.9 constraint does allow it. Therefore the Laravel framework pulls in the latest version of brick/money (0.10.x) which is too high for brick/money ^0.5.3.

It is possible to run composer update -W initbiz/money-plugin which will downgrade the brick/math dependency to a lower version which can be used by both this plugin and ramsey/uuid. However, this is undesirable because it installs an older version of the package.

Is there any specific reason this package depends on an older brick/money library?

This plugin does not seem to work with the \October\Rain\Database\Traits\Encryptable trait. I've added the amount column to the encryptable array but the value is not encrypted in the database.

This repository seems to be missing a license declaration. Is it omitted on purpose? The plugin page says MIT but I feel this should also be reflected in the repository.