yes, it's in ModuleRepository which is being inherited by the repository of your module

yeah.

My browser uses ID as a title field (it doesn't have a title field at all). I changed $titleColumnKey to my Presenter method, which work on index page, but not in browser

tried also playing with $browserColumns in the Controller without any luck

so... what to do to make the browser use the Presenter field? Or any other field that is not an ID 😄

@User what do you mean? that message is for me?

message for you is in your DM's and in #811936425858695201 🙂 But if you know how to solve my problem, I will appreciate

Hi. Found a bug when switching to tiptap editor. wysiwyg editor will not run into Toolbaroptions settings. Has anyone solved this problem?

Hey @User , I think this should do for both the index and the browser list, in your module's controller:

protected $titleColumnKey = 'id';

I think we are not on the same page 😄 My Order form has OrderItems browser. The browser shows OrderItems with ID's instead of title (which the module does not have, so I understand this). I want the browser to use OrderItemsPresenter function instead of ID"s. I was able to make it work for Index page, but the browser is still showing ID's

Ok, I misunderstood. But what is the end goal actually, do you want to use a custom value instead of the raw ID value?

yes... sorry, I didn't express my intention correctly. The OrderItem has a Product relationship and I want to show product's name for Index and for the browser too

As I said, for Index I solved it with the presenter

php

class OrderItemPresenter extends Presenter {

    public function productName()
    {
        return $this->entity->product()->first()->name;
    }
}

not sure I followed, but try the

getTitleInBrowserAttribute

accessor maybe?

I think you could use an attribute accessor on the model, ie.

getCustomTitleAttribute()

and return your computed value. Then use

custom_title

as

$titleColumnKey

This worked, thanks... didn't know about that

not sure if I understand your solution. I have relationships like this: Order 1:n OrderItems n:1 Product. The Order form needs to show all the OrderItems in browser, but because OrderItem does not have anything that could be used as a name or title, I want the OrderItem in the browser to be represented by name of the Product

ok, I tried it but I'm confused af about how this works

This is what I had in mind — OrderItem.php:

public function getCustomTitleAttribute()
{
    return $this->product->title;
}

OrderItemController.php:

protected $titleColumnKey = 'custom_title';

I guess I just don't tend to reach for presenters all that often. Many ways to approach things.

it works for Index, but not for Browser. It has the same effect as using Presenter

Just did a quick test here on a fresh module and it worked. Not sure if some other config could have an influence in the browser's list.

yeah... I did things in getFormFieldsForBrowser and that's why it was using ID.... because I wrote it there. Tried to put there the name of the Presenter function but no success, I guess browsers can't use presenters

so both of your solution works but not mine with the Presenter.... 😄

is it possible to add target option on a Quill link element ? 😮

with Quill, target blank should automatically be added on links to external domains, but obviously that's not enough. one way I've seen it done at rendering time:

php
function fixReverseTabnabbingVulnerabiltyInQuill($html)
 {
    return str_replace(['target="_blank"', "target='_blank'"], 'target="_blank" rel="noopener"', $html);
}

How can I check if model is draft ?

@Luís Novais in which context? if the published property is false most likely

tiptap

yes I can I get that, I try ->published() but return true always if is in draft too

this doesn't sound right. published() is doing a where query on the published column to check if it's true. It's a scope that you call within a query. If you are working with a single record, you need to check

published

not

published()

. If you're actually trying to query only published records and getting records that have published set to false in the database, something's wrong somewhere else