Log message #4199775

# At Username Text
# Aug 23rd 2019, 16:52 mogpusse Hello
# Aug 23rd 2019, 16:16 admad @davorminchorov unless you really want the mail to travel through the interwebs, you can use something like mailhog on local network
# Aug 23rd 2019, 16:14 admad We should fix that in 3.next
# Aug 23rd 2019, 16:06 dereuromark I would say, yes
# Aug 23rd 2019, 14:43 rchavik it should by default escape the title, isn't it?
# Aug 23rd 2019, 14:36 dereuromark might be historically
# Aug 23rd 2019, 14:35 rchavik BreadcrumbsHelper::add() does not auto escape?
# Aug 23rd 2019, 14:05 davorminchorov I am mainly setting this up for mailtrap for local testing because the rate limit is 3 emails per second, I think it would be fine for other SMTP providers like gmail etc.
# Aug 23rd 2019, 14:03 ricksaccous i dunno, but yeah it's possible
# Aug 23rd 2019, 14:03 ricksaccous or you could also just add taht metadata in and check against it to say worker hold on a bit
# Aug 23rd 2019, 14:02 ricksaccous oh cool then yeah you should be able to mess with the workers and worker timings to set the rates
# Aug 23rd 2019, 14:02 dereuromark if it fails it would just resend. But yeah, you could build in a bit of rate limiting.
# Aug 23rd 2019, 14:02 davorminchorov Yeah, I am currently using cakephp-queue by dereuromark, I'll see the config if it does have any rate limits
# Aug 23rd 2019, 14:01 ricksaccous by default it just attempts to send all emails at once i believe
# Aug 23rd 2019, 14:01 ricksaccous you'd have to set things up so the plugin handles email sending though
# Aug 23rd 2019, 14:00 ricksaccous using the plugin you can specify a rate
# Aug 23rd 2019, 14:00 ricksaccous @davorminchorov if you need something like that I'd recommend the Queue plugin
# Aug 23rd 2019, 13:59 davorminchorov Is there a rate limit configuration for emails in CakePHP?
# Aug 23rd 2019, 13:54 dereuromark I would also think that the route itself is enough in that case
# Aug 23rd 2019, 13:52 neon1024 Only advantage I can see is that the controller method tells me which methods it likes right in the code
# Aug 23rd 2019, 13:51 neon1024 i don’t have fallback routes in this prefix, so I’m guessing checking in the controller is redundant as the route just won’t match right?
# Aug 23rd 2019, 13:51 neon1024 Feels like I’m doing the same thing from different ends
# Aug 23rd 2019, 13:50 neon1024 If I specify an http method in my route as `->setMethods(['get'])` for example, do I need to check `$this->getRequest->allowMethod('get')` in my controller?
# Aug 23rd 2019, 13:34 admad shell classes are deprecated, replaced by command classes. It is planned to make commands lib standalone but markstory hasnt got around to it
# Aug 23rd 2019, 13:32 val @admad Hi, would it be possible to have a standalone 3.x shell classes library like standalone ORM https://github.com/cakephp/orm?
# Aug 23rd 2019, 12:49 admad if you absolutely want to support both uuid and int then make 2 fk fields and configure which field to use in association
# Aug 23rd 2019, 12:48 slackebot2 'length' => 64, ```
# Aug 23rd 2019, 12:48 slackebot2 ->addColumn('parent_asset_id', 'integer', [ 'null' => true, 'default' => null, ]) - ->addColumn('foreign_key', 'string', [ - 'null' => true, 'default' => null, 'length' => 36, + ->addColumn('foreign_key', 'integer', [ + 'null' => true, 'default' => null, ]) ->addColumn('model', 'string', [ 'null' => true, 'default' => null,
# Aug 23rd 2019, 12:48 rchavik solved: ``` diff --git a/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php b/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php index 2be8a93f..b1a24282 100644 --- a/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php +++ b/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php @@ -64,8 +64,8 @@ class AssetsInitialMigration extends AbstractMigration {
# Aug 23rd 2019, 12:47 admad hehe, it happens, something we get too focused trying to solve things one way
# Aug 23rd 2019, 12:47 rchavik :face_palm:
# Aug 23rd 2019, 12:47 rchavik it never occured to me to change the damn schema
# Aug 23rd 2019, 12:47 rchavik you know..
# Aug 23rd 2019, 12:46 admad ah, noble cause :slightly_smiling_face:
# Aug 23rd 2019, 12:45 rchavik yeah, this was trying to support integer and char(36) for uuids
# Aug 23rd 2019, 12:45 admad not sure how db db will handle if the value isn't quoted
# Aug 23rd 2019, 12:44 admad overriding field type might creating problems when saving data
# Aug 23rd 2019, 12:44 admad but best is to avoid creating unnecessary problems which you have to solve :slightly_smiling_face:
# Aug 23rd 2019, 12:43 rchavik noted, i will try to change the field
# Aug 23rd 2019, 12:43 admad you can try overriding the field type in table table schema in Table::_initalizeSchema()
# Aug 23rd 2019, 12:41 admad *same as