Unable to save data on model with non numeric key about laraform HOT 6 CLOSED

Yes, this Hash library can only encode/decode numeric keys, which indeed aims to provide some level of security in production. It will not work with non-numeric keys though, but I'll come up with a solution for that in a later release, thanks! In the meantime I can recommended setting secret to false on guid, which should skip the encoding part.

from laraform.

Setting secret to false, I had already tried, does not work, the logic still enters setKeyFormData() and in there it correctly enters the if where it attempts the Hash:decode($key); which returns an empty array.

My workaround works though (!?)

from laraform.

I've changed my workaround, left ln 814 at Laraform.php as it was, and changed ln 28 of Support/Hash.php from;

return (new Hashids(md5(env('APP_KEY')), 10))->decode($value)[0]

to

return (new Hashids(md5(env('APP_KEY')), 10))->decode($value)[0] ?? $value;

from laraform.

If your fix works, it's fine, but I'll look into the issue anyway. A whole example with model and migration would help though.

from laraform.

Table looks like;

CREATE TABLE objects (
guid char(40) COLLATE utf8mb4_unicode_ci NOT NULL,
name varchar(255) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
type varchar(20) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
status varchar(20) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
_props longtext COLLATE utf8mb4_unicode_ci DEFAULT NULL,
UNIQUE KEY objects_guid_unique (guid)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;

Model has nothing special, looks like;

namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Arr;
use Laravel\Scout\Searchable;

class Obj extends Model
{
use Searchable;
//
protected $connection;
protected $table = 'objects';
protected $primaryKey = 'guid';
protected $keyType = 'string';
protected $casts = [
'_props' => 'json',
];
public $incrementing = false;
}

from laraform.

Hi! I've created a new bug report template, so if you still have the issue please resubmit accordingly: https://github.com/laraform/laraform/issues/new?assignees=&labels=bug&template=1_bugs.md&title=%5BBUG%5D

(was not able to reproduce the bug because I was thrown "Unknown column" sql error)

from laraform.