@lelkneralfaro

elkaro → created an issue.

@ptmkenny Yes I am using it to encrypt some node fields.

@ptmkenny unfortunately I don't know any more than I put in my comment. When I ran drush updb and src/FieldEncryptServiceProvider.php was present, I got the error, if I temporarily removed the file I could run drush updb successfully.

On Drupal 11.1.4.

I am using commit `ef79e5b425c` of oop_hooks_workaround4

And drush updb returns this error:

   [error]  Error: Call to a member function insert() on null in Drupal\Core\Lock\DatabaseLockBackend->acquire() (line 71 of /var/www/html/web/core/lib/Drupal/Core/Lock/DatabaseLockBacken  
  d.php) #0 /var/www/html/web/core/lib/Drupal/Core/Cache/CacheCollector.php(235): Drupal\Core\Lock\DatabaseLockBackend->acquire()                                                            
  #1 /var/www/html/web/core/lib/Drupal/Core/Cache/CacheCollector.php(312): Drupal\Core\Cache\CacheCollector->updateCache()                                                                   
  #2 /var/www/html/web/core/lib/Drupal/Core/DrupalKernel.php(691): Drupal\Core\Cache\CacheCollector->destruct()                                                                              
  #3 /var/www/html/vendor/drush/drush/src/Boot/DrupalBoot8.php(312): Drupal\Core\DrupalKernel->terminate()                                                                                   
  #4 [internal function]: Drush\Boot\DrupalBoot8->terminate()                                                                                                                                
  #5 {main}. 

No idea what is going on, but poking around through intuition and git bisect I found that src/FieldEncryptServiceProvider.php causes the error, and deleting that file makes the error go away, although likely causes other problems because I imagine it is there for a reason.

   [error]  Error: Call to a member function insert() on null in Drupal\Core\Lock\DatabaseLockBackend->acquire() (line 71 of /var/www/html/web/core/lib/Drupal/Core/Lock/DatabaseLockBacken  
  d.php) #0 /var/www/html/web/core/lib/Drupal/Core/Cache/CacheCollector.php(235): Drupal\Core\Lock\DatabaseLockBackend->acquire()                                                            
  #1 /var/www/html/web/core/lib/Drupal/Core/Cache/CacheCollector.php(312): Drupal\Core\Cache\CacheCollector->updateCache()                                                                   
  #2 /var/www/html/web/core/lib/Drupal/Core/DrupalKernel.php(691): Drupal\Core\Cache\CacheCollector->destruct()                                                                              
  #3 /var/www/html/vendor/drush/drush/src/Boot/DrupalBoot8.php(312): Drupal\Core\DrupalKernel->terminate()                                                                                   
  #4 [internal function]: Drush\Boot\DrupalBoot8->terminate()                                                                                                                                
  #5 {main}. 

I'm glad to have found this issue. I ran into the same problem today.

I'm going to reopen the issue in case there is anything that can be done on this module's side.

mlncn → credited elkaro → .

I lied, tomselect was not the problem, the issue was that the data export display filter was using Autocomplete and the page filter was using Dropdown

elkaro → created an issue.

elkaro → created an issue.

Working as expected for me. Looks great!

I agree with @guido_s. The NodePermissions class was added on 9.4 ( Change record → )
So a version of the module specifying that as the minimum core requirement would satisfy the requirements for this issue fork to be merged.

Switching to Needs work, but I think this is RTBC once a release with the updated core version requirement comes out.

elkaro → created an issue.

I had double-quotes in my "Views query" output. I needed to remove the curly braces and the double quotes in order to run the sql query.

Just a heads up to others who may  find themselves here

This issue is actually with entity_browser and has been discussed although not resolved here:
https://www.drupal.org/project/entity_browser/issues/2853751 →

lelkneralfaro → created an issue.

I ran testBlockAccess locally and it passed. The CI says that it failed:

https://git.drupalcode.org/issue/drupal-3029830/-/jobs/3186760

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

If it is of any help I found a recursive element that causes json_encode to fail:

$props_raw['rows'][0]['cells'][1]['content']['source']['#groups']['advanced'][0]['alias']['#groups']);

With '#groups' recurring forever

The code in the issue fork only works with code I have added in this sandbox module: https://git.drupalcode.org/sandbox/lelkneralfaro-3467019

So, if Smart Date maintainers are willing for this feature to be fully developed in smart date I would move what I have begun on in the sandbox module into this issue fork.

As far as I can tell for this feature to be implemented as an independent module there would have to be these additions to Smart Date which is why I opened this issue fork.

lelkneralfaro → created an issue.

Thanks. The indentation was added to issue fork merge commit.

lelkneralfaro → made their first commit to this issue’s fork.

Thanks for the questions. Here are the answers:

1. Yes, the alter hook worked on 10.2 without problems
2. Here is the alter hook:

function my_module_language_switch_links_alter(array &$links, $type, \Drupal\Core\Url $url) {
  $current_language = \Drupal::languageManager()->getCurrentLanguage()->getId();
  unset($links[$current_language]);
}

3. Here is the error and stack trace:

The website encountered an unexpected error. Try again later.

Symfony\Component\Routing\Exception\RouteNotFoundException: Route "" does not exist. in Drupal\Core\Routing\RouteProvider->getRouteByName() (line 208 of core/lib/Drupal/Core/Routing/RouteProvider.php).

Drupal\Core\Routing\UrlGenerator->getRoute('') (Line: 132)
Drupal\Core\Routing\UrlGenerator->getPathFromRoute('', Array) (Line: 71)
Drupal\Core\Render\MetadataBubblingUrlGenerator->getPathFromRoute('', Array) (Line: 808)
Drupal\Core\Url->getInternalPath() (Line: 137)
Drupal\Core\Utility\LinkGenerator->generate('English', Object) (Line: 102)
Drupal\Core\Render\Element\Link::preRenderLink(Array)
call_user_func_array(Array, Array) (Line: 113)
Drupal\Core\Render\Renderer->doTrustedCallback(Array, Array, 'Render #pre_render callbacks must be methods of a class that implements \Drupal\Core\Security\TrustedCallbackInterface or be an anonymous function. The callback was %s. See https://www.drupal.org/node/2966725', 'exception', 'Drupal\Core\Render\Element\RenderCallbackInterface') (Line: 870)
Drupal\Core\Render\Renderer->doCallback('#pre_render', Array, Array) (Line: 432)
Drupal\Core\Render\Renderer->doRender(Array) (Line: 504)
Drupal\Core\Render\Renderer->doRender(Array, ) (Line: 248)
Drupal\Core\Render\Renderer->render(Array) (Line: 475)
Drupal\Core\Template\TwigExtension->escapeFilter(Object, Array, 'html', NULL, 1) (Line: 182)
__TwigTemplate_750efad714b8da68ad259b30d7ec4b74->doDisplay(Array, Array) (Line: 360)
Twig\Template->yield(Array) (Line: 335)
Twig\Template->render(Array) (Line: 38)
Twig\TemplateWrapper->render(Array) (Line: 33)
twig_render_template('themes/contrib/gin/templates/dataset/table.html.twig', Array) (Line: 348)
Drupal\Core\Theme\ThemeManager->render('table', Array) (Line: 491)
Drupal\Core\Render\Renderer->doRender(Array) (Line: 504)
Drupal\Core\Render\Renderer->doRender(Array) (Line: 504)
Drupal\Core\Render\Renderer->doRender(Array, ) (Line: 248)
Drupal\Core\Render\Renderer->render(Array, ) (Line: 238)
Drupal\Core\Render\MainContent\HtmlRenderer->Drupal\Core\Render\MainContent\{closure}() (Line: 638)
Drupal\Core\Render\Renderer->executeInRenderContext(Object, Object) (Line: 239)
Drupal\Core\Render\MainContent\HtmlRenderer->prepare(Array, Object, Object) (Line: 128)
Drupal\Core\Render\MainContent\HtmlRenderer->renderResponse(Array, Object, Object) (Line: 90)
Drupal\Core\EventSubscriber\MainContentViewSubscriber->onViewRenderArray(Object, 'kernel.view', Object)
call_user_func(Array, Object, 'kernel.view', Object) (Line: 111)
Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher->dispatch(Object, 'kernel.view') (Line: 186)
Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object, 1) (Line: 76)
Symfony\Component\HttpKernel\HttpKernel->handle(Object, 1, 1) (Line: 53)
Drupal\Core\StackMiddleware\Session->handle(Object, 1, 1) (Line: 48)
Drupal\Core\StackMiddleware\KernelPreHandle->handle(Object, 1, 1) (Line: 28)
Drupal\Core\StackMiddleware\ContentLength->handle(Object, 1, 1) (Line: 32)
Drupal\big_pipe\StackMiddleware\ContentLength->handle(Object, 1, 1) (Line: 106)
Drupal\page_cache\StackMiddleware\PageCache->pass(Object, 1, 1) (Line: 85)
Drupal\page_cache\StackMiddleware\PageCache->handle(Object, 1, 1) (Line: 48)
Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle(Object, 1, 1) (Line: 51)
Drupal\Core\StackMiddleware\NegotiationMiddleware->handle(Object, 1, 1) (Line: 36)
Drupal\Core\StackMiddleware\AjaxPageState->handle(Object, 1, 1) (Line: 51)
Drupal\Core\StackMiddleware\StackedHttpKernel->handle(Object, 1, 1) (Line: 741)
Drupal\Core\DrupalKernel->handle(Object) (Line: 19)

For further context, the issue is also related to me unsetting the current language in a hook_language_switch_links_alter(). If I do not unset the current language the problem disappears, but then the current language is visible which is what I wanted to avoid.

lelkneralfaro → created an issue.

This patch takes care of it and hopefully has the correct schema structure.

lelkneralfaro → created an issue.

Having this feature for the 2.0.x version of the module would be a blessing. I have a large menu that needs translating and this would be a tremendous help.

I am not sure what happened the first time around, but after another clean site install today I found the 'date occurrence' field and was able to add it and use it as a filter. Thanks for the help!

I imagine I might see the field here:

Thanks for the screenshot. Unfortunately that field is not available for me in the fields list.

To verify that I have set up everything correctly here is what I have done:
- Create a content type (I have called it event) with a 'Recurring dates field' (see attached screenshot ending in 09-25-34)
- Add a Search API Server using Search API's database backend
- Add an index to that server called 'Test Events'
- Add 'Content' and 'Data occurrences' datasources to the 'Test Events' index. (see screenshot ending in 09-27-47)

When I try to add fields I do not see the 'Date occurrence' field available. Or what in my case would be labeled 'Repeating date occurrence' since my field is labeled differently.

No luck finding a 'Date occurrence' field. I have the 'Date occurrences' datasource but no 'Date occurrence' field on my 'Content' data source nor on the 'Date occurrences: Content (field_repeating_date)' data source.

lelkneralfaro → created an issue.

That fixed it, thanks!

I understand. Thanks for the quick response, consideration of the request and the advice.

If you do end up being interested in being sponsored to to create a Smart Date Recur Search API module just let me know.

I also encountered this issue. After looking into it I see that the alpha9 release which is the latest release still has a bool return type.

The dev branch has the change to void.

Until Computed Fields has a new release I'll try out using the dev branch.

lelkneralfaro → created an issue.

Uploading new patch due to error in date formatting in previous patch showing month rather than hour for end date.

This feature is very helpful to me. It would be great to get a release some time soon. Thanks.

smustgrave → credited lelkneralfaro → .

Great. Thank you for the quick response.

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

I tested your MR and the fix worked. Marking RTBC. Thank you!

Apologies for my delay on reviewing this. I will review today.

lelkneralfaro → created an issue.

Make the field a 'date range' field

lelkneralfaro → created an issue.

I haven't looked into the code enough to figure out what might be going on and suggest a solution so I just wanted to report an issue/observation based on my use of the module.

Background:
I am currently using search_api_saved_searches version 8.x-1.0-rc1

I have been using the 'Default' search type created from installation of the module over a year ago.

Observation:

The notification interval field used to be available on the 'Default' search type but no longer is. When I create a new search type, 'notification interval' is available as a form field.

Suggested solution (to take with a grain with salt):

Maybe an update hook is required to update configuration for already created search types.

lelkneralfaro → created an issue.

I have the same need of adding counties as well as states. I have reviewed the link included in the description of this issue and based on that documentation it looks to me like they are describing how to modify the administrative_area subfield, not add a new subfield, which is what I need.

How would I go about adding a county subfield?

I encountered a case where the help text was important when deleting a node so this would be a helpful feature.

Included patch adds spacing when rendered:

And removes stacking on mobile:

Sorry about all the noise adding the compiled css generates, but including so others do not have to compile themselves. The important bit is in _octavia.scss

lelkneralfaro → created an issue.

Completed in commit 5fb4cdc0

Thanks for the suggestion. Yes that could be a good place for it.

Just started with the issue fork here so that the community is aware if there is any interest.

lelkneralfaro → created an issue.

@Taran2L https://git.drupalcode.org/project/nodeviewcount/-/merge_requests/2/diff... removes the composer.json file which is needed to directly include the issue fork in a project's root composer.json.

Was there a reason for dropping the composer.json? If not, please bring it back. Thanks.

lelkneralfaro → created an issue.

Patch #2 is the wrong approach. Instead we are switching to use responsive images:

Ref commit: 6fe28ba7

lelkneralfaro → created an issue.

Replaced all 'field_image' with 'field_media_image' in templates.

For 'paragraph--slide.html.twig' templates and all its skin derivatives I have used the method below for getting the image url but if upon review there is a better method please provide an updated patch.

{% set image_url = file_url(content.field_media_image[0]['#item'].entity.getFileUri()) %}

lelkneralfaro → created an issue.

This patch was against 8.x-2.6 and the fix was released in 8.x-2.8 so I'll close this issue.

Issue 🐛 Drupal\Core\File\Exception\NotRegularDirectoryException: %app.root/modules/contrib/gutenberg is not a directory Fixed is indeed where it was fixed.

lelkneralfaro → created an issue.

lelkneralfaro → made their first commit to this issue’s fork.

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

This patch seems to do the trick. It seems that when an calendar event's start date is not 'multiple' the lookup key for a calendar event was getting the wrong event and then assigning the entry's event dates based on the wrong event.

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

I would be happy to take on co-maintainership :)

Fixed issue https://www.drupal.org/project/split_preview/issues/3377253 🐛 "Close" and "Fullscreen" buttons do not work. Active in this issue fork.

with this commit:
https://git.drupalcode.org/issue/split_preview-3289816/-/commit/ce8cc327...

Fixed in https://www.drupal.org/project/split_preview/issues/3289816 📌 Automated Drupal 10 compatibility fixes Fixed issue fork

lelkneralfaro → created an issue.

Added documentation to cover new feature of module.

Package added on packagist!
https://packagist.org/packages/drupal/drutopia_dev_template

Thanks!

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

lelkneralfaro → created an issue.

Patch applied to 2.0.x-dev branch

lelkneralfaro → made their first commit to this issue’s fork.

joegraduate → credited lelkneralfaro → .