Los Angeles
Account created on 17 February 2007, almost 18 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States loze Los Angeles

@piyusha That is a core bug thats being worked on here 🐛 Nested modals don't work: opening a modal from a modal closes the original Needs work It's not specific to gutenberg

🇺🇸United States loze Los Angeles

Never mind. I see now that this is something being added to core.

🇺🇸United States loze Los Angeles

loze created an issue. See original summary .

🇺🇸United States loze Los Angeles

Apparently ckeditor is not triggering Drupal.attachBehaviors() when its rendering media items. So I guess its not an issue for this module.

🇺🇸United States loze Los Angeles
🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

Created a MR from #11 I have been using this patch for quite some time as well.

🇺🇸United States loze Los Angeles

@tr, I just left a comment on that issue and marked it as RTBC. I cate about this module and currently use it on two sites and am happy to help you get a stable release out.

🇺🇸United States loze Los Angeles

I have been using this MR for a few months and everything appears to work as expected.

🇺🇸United States loze Los Angeles

Just needed to include core/internal.jquery.form as a dependency in the .yml

MR4 does the trick

🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

I am seeing the same as @malcomio and this patch fixes it for me as well.
Drupal 10.4.2

🇺🇸United States loze Los Angeles

Tested. fixes the issue. thanks!

🇺🇸United States loze Los Angeles

The patch in #149 applies and works as expected for me in drupal 10.4
thanks!

🇺🇸United States loze Los Angeles

I've done some work here to improve the drupalblock config form in MR214 and its working for me.

The form now submits with the core drupal ajax and runs the block through the validation and submit handlers.

I borrowed a lot of this from the context modules block reaction code. That module is similar to what we are doing here, in that it allows you to configure blocks via an ajax form outside of the core block layout UI. The main difference is the context module stores the configs in actual config files while we are storing the config data in the body of the content as json.

you can see how the context module handles the bock config form here https://git.drupalcode.org/project/context/-/blob/5.x/src/Reaction/Block...

This solves my initial issue of drupal blocks config fields that use ckeditor not saving values.

This also solves the issue here Drupal block - Change and/or hide title Active because now we are including the whole block config form. It also allows other settings like items per page in views blocks and any 3rd part settings alters that were not working.

🇺🇸United States loze Los Angeles

Yea, I think a modal is a better approach.

🇺🇸United States loze Los Angeles

I think this needs to be handled similar to what I implemented for content blocks 🐛 Libraries attached to content block fields are not included in the editor Fixed using drupal's core ajax commands.

I will try to write a patch/MR

Also I think this is related Drupal block - Change and/or hide title Active

🇺🇸United States loze Los Angeles

Tested the 3.0 version. and its working, thanks

🇺🇸United States loze Los Angeles

Its worth mentioning that the issue described in #8 happens w/o this patch as well. If the block is in the sidebar.

There should be some way to block clicking on another item, or to autosave the the block if they do.

🇺🇸United States loze Los Angeles

MR20 allows me to cast votes on my fields again.

I know @tr wants tests for this sort of stuff, however thats not something I know how to do. I've tried to figure it out countless times for various projects but cant seem to wrap my head around the concept of writing tests. Hopefully someone else can help out with that.

🇺🇸United States loze Los Angeles

+1 can someone commit this?

🇺🇸United States loze Los Angeles

I understand, but thats the thing, there are no errors in my php error logs when this happens.

🇺🇸United States loze Los Angeles

The error does not happen with 1.4 or 1.x-dev

🇺🇸United States loze Los Angeles

Should we be making this feature for 3.x now?

It looks like 3.0 already handles free orders with a new setting for require_payment_method to 'Collect payment methods on orders with zero balance'

🇺🇸United States loze Los Angeles

This MR allows this module to work with D10 and commerce 3.0

Can someone create a release with it?

🇺🇸United States loze Los Angeles

This works fairly well in many cases, however when the IEF has a media browser field in it, when you click the button to open the media browser the entity embed IEF modal is replaced with the media browser, and clicking the insert media button closes the whole modal so nothing is ever added.

Clicking a media browser button (or any for element that opens a modal) should spawn a new modal on top. Im not sure if this is something to address here or with the media module.

🇺🇸United States loze Los Angeles

Can someone reroll this for 10.4.1 ?

🇺🇸United States loze Los Angeles

I tested, and see the same as @smustgrave

The correct icon is used in ckeditor when editing content but it is missing on the input filter config form.

🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

I created MR16 from the patch in #37 to help move this along.

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2927167-autocomplete-support-does to hidden.

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2927167-autocomplete-support-does to active.

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2927167-autocomplete-support-does to hidden.

🇺🇸United States loze Los Angeles

Tested. The patch applies and works as advertised.

+1 for this

🇺🇸United States loze Los Angeles

MR125 Adds a setting in the field widget for avatar image style.

🇺🇸United States loze Los Angeles

I have a working patch I will submit

🇺🇸United States loze Los Angeles

Tested with alpha2 and dev and it seems to do the trick. Thanks!

🇺🇸United States loze Los Angeles
🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

Tested. That does fix the issue. no changes to realname are needed with this approach. Thanks.

🇺🇸United States loze Los Angeles

Yes, MR!175 does appear to do the job with version 4.0 alpha

thanks!

🇺🇸United States loze Los Angeles

Thanks. Since the original post, this module now has its own entity reference selection plugin instead of using a direct query. So I created an issue in realname to support private_message which is just a small change and it appears to be working with the 3.0 branch.

Private Message module support Active

🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

I believe this is the same issue being described in 🐛 Sending private message to more than one recipient does not work Needs work and I believe started with Drupal 10.3

The issue appears when adding more than one user/recipient to a private message.

When adding the 2nd recipient, this modules js is targeting the "add more" button incorrectly. The js attempts to simulate a click on the add more button, but instead targets the first "remove" button of the hidden autocomplete field.

when this was first written core did not have a remove button on the autocomplete field.

🇺🇸United States loze Los Angeles

Patch #26 was no longer applying to the the latest dev version, so I've created MR14 with the changes in #26

here is a patch to use for composer

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 8.x-3.x to hidden.

🇺🇸United States loze Los Angeles

loze made their first commit to this issue’s fork.

🇺🇸United States loze Los Angeles

I have a D10 version of commerce_affiliate and affiliate_ng 95% working that I want to contribute.

Can the maintainers create a 2.x branch of each product so I can provide a MR for others to test out?

Or whats the best way for me to add it?

🇺🇸United States loze Los Angeles

I spent a little time and have most of this working locally. Should I add a MR or create a new module?

I don't know if I can commit to maintaining it long term but I'm happy to help get it up and running.

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2919646-is-there-a to hidden.

🇺🇸United States loze Los Angeles

I also need this for a project I'm currently working on.

Has anyone started any work on a D10 version? I can help code and test?

🇺🇸United States loze Los Angeles

Sorry, this was my mistake. disregard.

🇺🇸United States loze Los Angeles

MR!2 adds a batch process to hook_install() and allows me to install the module.

Here is a patch of the MR for composer.

🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

loze created an issue.

🇺🇸United States loze Los Angeles

@claudiu.cristea I dont believe that getDisplayName() is a queryable field, right?

I'm trying to get the auto complete search to use the compiled display name that is stored in the realname table.

🇺🇸United States loze Los Angeles

Thanks for the test @agile-mark-l this looks good.

🇺🇸United States loze Los Angeles

I added a template for the loading animation quicktabs-loading.html.twig so you can override this in your theme, per qt instance if needed.

$markup = [
              '#theme' => 'quicktabs_loading',
              '#message' => t('Loading content ...'),
              '#instance' => $qt_instance,
              '#tabid' => $tab_id,
            ];

Attached is a patch from !MR21 for composer.

🇺🇸United States loze Los Angeles

Resolved the conflicts in MR36

heres a patch for composer

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2.1.x to hidden.

🇺🇸United States loze Los Angeles

MR!122 Adds this functionality. Here is a patch for composer.

🇺🇸United States loze Los Angeles

This is fixed in the latest version

🇺🇸United States loze Los Angeles

To clarify, I'm trying to do two things here:

1. display the users picture/avatar next to their name in the autocomplete dropdown
2. alter the autocomplete query to further limit who someone is allowed to send a message to (for example only someone in a friends list)

I figure the best way to do this is to use a custom view to supply the results of the autocomplete dropdown, but not sure the best way to go about doing this.

any suggestions?

🇺🇸United States loze Los Angeles

Last patch in #22 was wrong, some unintended changes made it in there.
this one should work with composer

🇺🇸United States loze Los Angeles
🇺🇸United States loze Los Angeles

Its been some time since the original issue and patches.

Since then this module now uses the users display name instead of the username, which resolves 1/2 of what was being addressed in the original patches.

I have created MR!121 which adds realname support to the autocomplete query for the members widget if realname is installed.

here is a patch for the MR to use with composer.

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2987189-realname-support to active.

🇺🇸United States loze Los Angeles

loze changed the visibility of the branch 2987189-realname-support to hidden.

🇺🇸United States loze Los Angeles
🇺🇸United States loze Los Angeles

here is a patch for composer

🇺🇸United States loze Los Angeles

Patch #35 does not work as pointed out in #38

Patch #32 does work which is a diff for MR40. Please test against those.

🇺🇸United States loze Los Angeles

MR7 gets this working for me.

🇺🇸United States loze Los Angeles

this latest change gives me this error

Error: Call to a member function moduleExists() on null in Drupal\soembed\Plugin\Filter\SoEmbedFilter->getEnabledProviders()

Production build 0.71.5 2024