Hello and thanks for the module

It would be very useful to be able to change a node language. There is no module now that lets us do it. Thanks

CommentFileSizeAuthor
#4 2993847-4.patch1.37 KBel1_1el
#2 2993847-2.patch1.18 KBel1_1el

Comments

candelas created an issue. See original summary.

el1_1el’s picture

el1_1el commented
Status: Active » Needs review
StatusFileSize
new 2993847-2.patch1.18 KB

Well it seems I am excluding ALL basefields, when I should probably only exclude some (ie nid, uuid, etc.)

Give this patch a try and I'll commit to dev if its working for you.

el1_1el’s picture

el1_1el commented
Status: Needs review » Needs work

on further testing, default_langcode cannot be changed and dates need reformatting. langcode does seem to work however. will do a new patch shortly

el1_1el’s picture

el1_1el commented
Status: Needs work » Needs review
StatusFileSize
new 2993847-4.patch1.37 KB

removed the base date fields and added a TODO for those. This should at least add the feature requested. Let me know. Thanks!

candelas’s picture

candelas commented

Hello and thanks

I tested your patch in #4 and by one side, good news because it changes the language, but I got 

Deprecated function: Non-static method Drupal\bulk_update_fields\BulkUpdateFields::bulkUpdateFieldsFinishedCallback() should not be called statically in _batch_finished() (line 456 of core/includes/batch.inc).
_batch_finished() (Line: 98)
_batch_page(Object) (Line: 55)
Drupal\system\Controller\BatchController->batchPage(Object)
call_user_func_array(Array, Array) (Line: 123)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}() (Line: 582)
Drupal\Core\Render\Renderer->executeInRenderContext(Object, Object) (Line: 124)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext(Array, Array) (Line: 97)
Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}() (Line: 151)
Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object, 1) (Line: 68)
Symfony\Component\HttpKernel\HttpKernel->handle(Object, 1, 1) (Line: 57)
Drupal\Core\StackMiddleware\Session->handle(Object, 1, 1) (Line: 47)
Drupal\Core\StackMiddleware\KernelPreHandle->handle(Object, 1, 1) (Line: 99)
Drupal\page_cache\StackMiddleware\PageCache->pass(Object, 1, 1) (Line: 78)
Drupal\page_cache\StackMiddleware\PageCache->handle(Object, 1, 1) (Line: 47)
Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle(Object, 1, 1) (Line: 52)
Drupal\Core\StackMiddleware\NegotiationMiddleware->handle(Object, 1, 1) (Line: 23)
Stack\StackedHttpKernel->handle(Object, 1, 1) (Line: 666)
Drupal\Core\DrupalKernel->handle(Object) (Line: 19)

Also I got the message

    All fields were updated successfully
    0 operations processed.

I see that the update date in Views page doesn't change.

Thanks for your generous work :)

el1_1el’s picture

el1_1el commented

Are you using the dev branch? That bug should be fixed in dev. see https://www.drupal.org/project/bulk_update_fields/issues/2919552

candelas’s picture

candelas commented

Hello and sorry

I was in a hurry and didn't realize that it was not the dev version, nor I search for the error in the issue queue.
I have just done it with the patch and dev. When I made the update for the bbdd, I got this error in 8001 update

Error: Drupal\Component\Plugin\Exception\PluginNotFoundException: The "bulk_update_fields_action_base" plugin does not exist. en Drupal\Core\Plugin\DefaultPluginManager->doGetDefinition() (linea 52 de /xxx/web/core/lib/Drupal/Component/Plugin/Discovery/DiscoveryTrait.php).

After that, I tested the module and not warnings or errors, and the module works perfect.

Thanks very much for your time :)

  • el1_1el committed e4bf0ab on 8.x-1.x 
    Issue #2993847 by el1_1el: Allow change node language
el1_1el’s picture

el1_1el commented
Status: Needs review » Patch (to be ported)

Thanks candelas!

I went ahead and committed this to dev as I did not get the db update error after installing alpha1 on a fresh d8.6 and then upgrading to dev with the patch. I'll leave it as to be ported for now. almost time for a second alpha.

thanks again!

candelas’s picture

candelas commented

I use D8.5.6 :)

el1_1el’s picture

el1_1el commented
Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.