Account created on 20 February 2017, about 8 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States keiserjb

I'm looking at the $results in the search method of SearchApiAiSearchBackend.php.

🇺🇸United States keiserjb

I needed this because I tried to test the 1.1.x-dev AI Agents on a site using s3fs. It need to use the FileSystemInterface. The patch doesn't apply because it looks like AgentHelper.php has already been altered in 1.1.x-dev. AiAgentBase.php has not though.

🇺🇸United States keiserjb

If anyone does want to get this to work with Lando. I had success with this Lando file.

name: drupal11-dev
recipe: drupal11

config:
  webroot: web
  php: '8.3'
  database: 'mysql:8.0'
  xdebug: false
  config:
    php: .lando.php.ini

services:
  appserver:
    overrides:
      environment:
        XDEBUG_MODE: 'debug,develop'
        PHP_IDE_CONFIG: "serverName=appserver"
        LANDO_HOST_IP: "host.docker.internal"
        XDEBUG_CONFIG: "remote_enable=1 remote_host=host.docker.internal"

  pma:
    type: phpmyadmin

  etcd:
    type: lando
    api: 3
    app_mount: false
    services:
      image: quay.io/coreos/etcd:v3.5.0
      command: >
        etcd -advertise-client-urls=http://127.0.0.1:2379
             -listen-client-urls http://0.0.0.0:2379
             --data-dir /etcd
      volumes:
        - ./docker/milvus/etcd:/etcd

  minio:
    type: lando
    api: 3
    app_mount: false
    services:
      image: minio/minio:RELEASE.2024-06-26T01-06-18Z
      command: minio server /minio_data
      ports:
        - "9000:9000"
        - "9001:9001"
      environment:
        MINIO_ACCESS_KEY: minioadmin
        MINIO_SECRET_KEY: minioadmin
      volumes:
        - ./docker/milvus/minio:/minio_data

  milvus:
    type: lando
    api: 3
    app_mount: false
    services:
      image: milvusdb/milvus:v2.4.5
      command: [ "milvus", "run", "standalone" ]
      ports:
        - "19530:19530"
        - "9091:9091"
      environment:
        ETCD_ENDPOINTS: etcd:2379
        MINIO_ADDRESS: minio:9000
        GRPC_PORT: 19530
      depends_on:
        - etcd
        - minio
      extra_hosts:
        - "milvus:127.0.0.1"


  attu:
    type: lando
    api: 3
    app_mount: false
    services:
      image: zilliz/attu:v2.4
      command: [ "node", "dist/src/app.js" ]
      ports:
        - "3000:3000"
      environment:
        MILVUS_URL: grpc://milvus:19530
    proxy:
      - attu-drupal11dev.lndo.site:3000

tooling:
  xon:
    service: appserver
    description: Enable Xdebug
    user: root
    cmd:
      - docker-php-ext-enable xdebug && kill -USR2 $(pgrep -o php-fpm) > /dev/null || /etc/init.d/apache2 reload
      - tput setaf 2 && echo "Xdebug On" && tput sgr 0 && echo

  xoff:
    service: appserver
    description: Disable Xdebug
    user: root
    cmd:
      - rm /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini && kill -USR2 $(pgrep -o php-fpm) > /dev/null || /etc/init.d/apache2 reload
      - tput setaf 1 && echo "Xdebug Off" && tput sgr 0 && echo

  milvus-status:
    service: appserver
    description: Check Milvus REST health
    cmd: curl http://milvus:9091/healthz

  open-attu:
    description: Open Attu interface in browser
    service: appserver
    cmd: echo "🎉 Attu is available at http://attu.drupal11-dev.lndo.site:3000"

🇺🇸United States keiserjb

It was my fault but it was possible to install AI 1.1 and ai_avb_provider_pinecone 1.0.

When I matched them it worked.

🇺🇸United States keiserjb

I somehow had 1.0 dev installed with AI dev and it didn't work.

🇺🇸United States keiserjb

It looks like there was a change to the Interface here. https://git.drupalcode.org/project/ai/-/commit/d92520c1e5c8e998ddc3d4082...

I will file issues in the other modules.

🇺🇸United States keiserjb

The new PR works for me.

🇺🇸United States keiserjb

I also had this problem. I fixed it in the https://www.drupal.org/project/ai_vdb_provider_pinecone module. I added QueryInterface $query, to the vectorSearch function.

🇺🇸United States keiserjb

Changing hook_library_info_build works for me as well with 1.1.16.

I had trouble replicating this problem after discovery last week as it still worked locally and on my dev site. Today, I was able to find out how to break Turnstile on my dev site by adding a domain. Then I got the cross-site cookie errors just like production. The js alterations fix it though.

🇺🇸United States keiserjb

I was getting cookie errors in the Chrome console.

Reading cookie in cross-site context may be impacted on Chrome
Cookies with the SameSite=None; Secure and not Partitioned attributes that operate in cross-site contexts are third-party cookies. Chrome is moving towards a new experience that allows users to choose to browse without third-party cookies.

Learn more from the linked article about preparing your site to avoid potential breakage.

It listed multiple cookies from the the cloudflare domain.

Thanks for the tip about Rocket Loader in Cloudflare. I turned that off and it started working.

🇺🇸United States keiserjb

The changes made to the admin form in the issue fork allow it to load successfully for me.

🇺🇸United States keiserjb

I got it to work with the changes.

🇺🇸United States keiserjb

I've been meaning to get back to this module and see if my latest changes are working. We open these issues in the modules we port just to make folks aware.

🇺🇸United States keiserjb

Changing the line in ckeditor5.ckeditor5.yml certainly fixes the problem.

I also tried adding the class in MediaLibrary.php. This worked too.

// Making the title for editor drupal media embed translatable.
    $static_plugin_config['drupalMedia']['dialogSettings'] = [
      'title' => $this->t('Add or select media'),
      'classes' => ['media-library-widget-modal'],
    ];
🇺🇸United States keiserjb

I had found another issue talking about this. I had noticed on my Drupal 11 site that the media library modal was narrower than in Drupal 10. I then found the missing class.

🇺🇸United States keiserjb

I found this other issue that discusses the same problem.

https://www.drupal.org/project/drupal/issues/3474018 🐛 [regression] Class is not added to MediaLibrary dialog Active

🇺🇸United States keiserjb

I have a local site on 10.3.10 and it works fine. I have another local site on 11.0.9 and the media-library-widget-modal class is not appearing.

🇺🇸United States keiserjb

I've noticed this problem as well. Trying to track it down.

🇺🇸United States keiserjb

Add TypedConfigManagerInterface to SiteAuditConfigForm constructor

- Updated constructor to inject TypedConfigManagerInterface as required.
- Fixed compatibility with ConfigFormBase parent class.
- Updated `create()` method to include `config.typed` service.

🇺🇸United States keiserjb

I see why there aren't any automated updates for Flexslider now. It looks like Drupal 11 moves to jquery 4. The flexslider library doesn't work with that and is no longer being developed.

🇺🇸United States keiserjb

I encountered a couple of other problems. In order to make it work I had to remove the ContainerInjectionInterface from DynamicPathProcessor.php.

I also needed to remove

calls:
- [setContainer, ['@service_container']]

from view_mode_page.services.yml

🇺🇸United States keiserjb

keiserjb created an issue.

🇺🇸United States keiserjb

Thank you, I didn't turn on my work computer over the weekend.

🇺🇸United States keiserjb

This version of the form works for me.

public function extraAdvancedFormFields(ContentEntityInterface $entity, FieldDefinitionInterface $fieldDefinition, FormStateInterface $formState, array $defaultValues = []) {
    $form = parent::extraAdvancedFormFields($entity, $fieldDefinition, $formState, $defaultValues);
    $settings = $fieldDefinition->getConfig($entity->bundle())->getSettings();

    $form['automator_clean_up'] = [
      '#type' => 'select',
      '#title' => $this->t('Text Manipulation'),
      '#description' => $this->t('These are possible text manipulations to run on each created tag.'),
      '#options' => [
        '' => $this->t('None'),
        'lowercase' => $this->t('lowercase'),
        'uppercase' => $this->t('UPPERCASE'),
        'first_char' => $this->t('First character uppercase'),
      ],
      // Use the default value from the third-party settings or fallback to a default.
      '#default_value' => $fieldDefinition->getConfig($entity->bundle())
        ->getThirdPartySetting('ai_automators', 'automator_clean_up', 'first_char'), // Ensure fallback here
      '#weight' => 23,
    ];

    $form['automator_clean_up'] = [
      '#type' => 'select',
      '#title' => $this->t('Text Manipulation'),
      '#description' => $this->t('These are possible text manipulations to run on each created tag.'),
      '#options' => [
        '' => $this->t('None'),
        'lowercase' => $this->t('lowercase'),
        'uppercase' => $this->t('UPPERCASE'),
        'first_char' => $this->t('First character uppercase'),
      ],
      // Use the default value from the third-party settings or fallback to a default.
      '#default_value' => $fieldDefinition->getConfig($entity->bundle())
        ->getThirdPartySetting('ai_automators', 'automator_clean_up', 'first_char'), // Ensure fallback here
      '#weight' => 23,
    ];

    if ($settings['handler_settings']['auto_create']) {
      $form['automator_search_similar_tags'] = [
        '#type' => 'checkbox',
        '#title' => 'Find similar tags',
        '#description' => $this->t('This will use GPT-4 to find similar tags. Meaning if the tag "Jesus Christ" exists and the system wants to store "Jesus" it will store it as "Jesus Christ". This uses extra calls and is slower and more costly.'),
        '#default_value' => $fieldDefinition->getConfig($entity->bundle())
          ->getThirdPartySetting('ai_automators', 'automator_search_similar_tags', 1), // Ensure fallback here
        '#weight' => 23,
      ];
    }

    return $form;
  }
🇺🇸United States keiserjb

Yep, it worked. Thanks.

🇺🇸United States keiserjb

Mine says "TypeError: t.Model is not a constructor". I think the patch resolved it.

🇺🇸United States keiserjb

keiserjb created an issue.

🇺🇸United States keiserjb

Based upon the behavior in here, how would any of us wish to become a customer?

🇺🇸United States keiserjb

Once again, I apoligize and thank you for the update.

🇺🇸United States keiserjb

I apologize for getting fired up but this is no longer an open source project if it is a paid service and the Drupal community cannot aid in the issue queue.

🇺🇸United States keiserjb

Why would I expect paid support to be any better than the support of the public project?

🇺🇸United States keiserjb

The community will help support this project if Opigno will let us. If not, expect a fork.

🇺🇸United States keiserjb

I think I've gotten this mostly working but I'm not releasing it yet. I pulled the latest dev branch to work on this. Thanks for the recent updates.

https://github.com/backdrop-contrib/scanner

🇺🇸United States keiserjb

So will Opigno be ready for Drupal 11? That's coming "soon" too.

🇺🇸United States keiserjb

Please let the community help move this project forward. It's been six months since Drupal 9 was deprecated. The development cycle is too slow, there are people who would help.

🇺🇸United States keiserjb

Could not agree more!

🇺🇸United States keiserjb

There are a lot of modules to fork. It is a pretty large undertaking.

🇺🇸United States keiserjb

I'll make a new issue with the altered methods if that is still a problem.

🇺🇸United States keiserjb

Very concerned about the status of Opigno and wondering about how the community could support this in some other way.

🇺🇸United States keiserjb

I have an implementation that adds a bee command that can be used on a new site install as well as a list that can be cut and pasted into the project installer.

🇺🇸United States keiserjb

Putting the Augmentor field inside the paragraph doesn't really work. Say I have a site without body fields. I only have paragraphs. If I put an Augmentor on a body paragraph to create the title of the node I can't target a field outside of the paragraph when configuring it.

🇺🇸United States keiserjb

I installed this patch on the dev branch of the module. An issue adding additional models was committed. https://www.drupal.org/project/augmentor_chatgpt/issues/3400720 Make available the gpt4 engine on the augmentor configuration Fixed

Because the method for getting the models is different for orhanderday and openai-php, the module breaks.

🇺🇸United States keiserjb

I found this thread because I'm trying to use the module with paragraphs. I could add the augmentor to the paragraph type but there may be more than one of the paragraph type and I would want the augmentor to use the data in all of the body paragraphs not just one of them.

Looking for ideas.

🇺🇸United States keiserjb

I installed this patch and it changed the icon colors to black. However, I could not seem to change the color to any other color but black. I thought this would make it possible to override the icon color in css.

🇺🇸United States keiserjb

Wondering about the progress on this. I watched the presentation from DrupalCon Lille and was confused about the capabilities of of these two modules. https://youtu.be/NdkME10H8MM?si=Xo1W8sHu7EsD5Rws

🇺🇸United States keiserjb

Looks like this works. I finally tried it.

🇺🇸United States keiserjb

Thanks for getting this fix up on here. There are lots of modules that don't include a description in the annotations. This patch does fix things. I couldn't add fields on multiple sites and now I can.

🇺🇸United States keiserjb

Rolling back wasn't really an option. Too much had happened. That being said, I updated all the tables in my database where the langcode was und. Everything works now.

🇺🇸United States keiserjb

I think I found it. The langcode for all of the migrated content was set to und. I discovered that new content was set to en. Changing the database for the langcode table fixes the display. I just wish there was an easier way to update the database.

🇺🇸United States keiserjb

I did the migration with the UI so there weren't any custom migration files. This might be a weird scenario with the content types being Features in D7 and then migrated.

🇺🇸United States keiserjb

I've found that if I create new nodes, the new ones show up in the report. The old ones that were migrated do not. Perhaps I can find what is happening and trick the old nodes into resaving correctly.

🇺🇸United States keiserjb

I'll look at it again next week. I don't recall any errors. The report just shows zeros for the counts when they are not supposed to be zero. I made a custom module this afternoon to display the node counts and it was all correct.

Something very well could have gotten scrambled during the migration. The content types not working were Features in D7 that were unfeaturized and migrated but they seem to work other than this report.

🇺🇸United States keiserjb

@joseph.olstad I'm not here to debate the merits of individual projects choosing to stay on D7, move to D10, move to B, or something else entirely. The fact is, after end of life this project and the project that shall not be mentioned will have the same common goal, which is keeping contrib projects and a core with very similar codebases secure. I know I would be willing to send along anything from the modules I've ported and maintain, however most of them are so simple that it's unlikely I'd help much. There has always been collaboration about security matters between B and D7 and I'm sure an extended security team would be no different.

🇺🇸United States keiserjb

@joseph.olstad

That issue is closed. I was just adding an FYI where people could find actual information about the other project. Please stop sending people to the issue you created.

🇺🇸United States keiserjb

I do not wish to get beaten down for mentioning Backdrop here, but a more relevant link about our project and community is the chat platform that we use. Just an FYI.

https://backdrop.zulipchat.com/

🇺🇸United States keiserjb

I've migrated a few sites from the WP to the B so if you want the Drupal stuff you're missing in WordPress...there is a path.

🇺🇸United States keiserjb

Got it. No problem. I love this module. I plan on using it as a starting point for a Backdrop module when I get around to it.

🇺🇸United States keiserjb

I had this problem on a site upgraded from D7 to D10. On the page with all of my patterns I happened to click on "Show row weights." They were all 0. I went back to "Hide row weights." I reordered some of them. Then I went back to "Show row weights" and they all had weights. I saved the list. Then the bulk alias generation worked.

🇺🇸United States keiserjb

Thanks, still getting the error when trying to install rc2 but not when using the dev branch.

🇺🇸United States keiserjb

I've had this problem as well. I added a Media Keywords field to my media types that references a taxonomy. I upload a media item and try to fill in terms. The autocomplete dropdown is behind the media library modal.

🇺🇸United States keiserjb

I think some of the confusion may be that the inline editor is still not working with ckeditor5.

🇺🇸United States keiserjb

I think it still needs to be

use Drupal\Component\Datetime\TimeInterface;

not use Drupal\Core\Time\TimeInterface;

🇺🇸United States keiserjb

It was because I had Media Bulk Upload and DropzoneJS installed.

🇺🇸United States keiserjb

This seems to work for me on the /media/add/image page but when I add a media item from a node edit page the alt text does not get added. Trying to figure out how to fix that.

🇺🇸United States keiserjb

Yes, this is on the roadmap for when I revisit the module next fall. I wanted to get this out before Halloween.

🇺🇸United States keiserjb

I thought that it might not have enough php after reading some comments on other projects. I tried this with another project which I may resurrect. Unfortunately this is my first module on d.org. I've only release modules in Backdrop before now. It looks like this approval will need to be delayed until I contribute to Drupal more.

Production build 0.71.5 2024