Account created on 27 January 2010, almost 15 years ago
#

Recent comments

🇮🇱Israel yakoub

//This means we may need to update access records,
// flush some caches containing the entity or perform other operations we
// cannot possibly know about.

if we go back to this comment, may i suggest to add new method to `Drupal\group\Plugin\Group\Relation\GroupRelationInterface`
which takes care of invoking exactly the kind of hooks required for each entity instead of calling $entity->save() ?

🇮🇱Israel yakoub

i am refactoring revision save logic, once that done i will fix the phpcs .

🇮🇱Israel yakoub

current 2.0.x branch is ready for testing .
do you require backporting to 1.0.x ?

🇮🇱Israel yakoub

current branch 2.0.x has working version, but :
given the eck route for adding content is shared among different entity types, then the method of defining tabs will not work when one entity has translations pack enabled while another entity is not even defined as translatable .
the solution instead of tabs is to add link for entity operations to dedicated add content route which loads the translations pack .
this should be simple solution, but given i have fulltime job then it will need to wait several days maybe until the weekend .

🇮🇱Israel yakoub

i refactored code to use entity handler, this way i can now continue to write custom handler for eck entities .
https://git.drupalcode.org/project/translations_pack/-/commit/a278a804df...

🇮🇱Israel yakoub

looks good .
i am also working on support for inline entity form .

🇮🇱Israel yakoub

it will be done this weekend .
it is just a routing derivative .

🇮🇱Israel yakoub

you are correct .
i am working on it .
the add new content route is defined in

src/Routing/TranslationsPackRouteSubscriber.php

i need to add support for eck or maybe define a new plugin type .

🇮🇱Israel yakoub

this module supports all entities .
simply enable content translation for your eck entities under path : admin/config/regional/content-language
and make sure to check "translations pack" like in the screenshot .
https://www.drupal.org/files/project-images/Screenshot%20from%202023-05-...

🇮🇱Israel yakoub

new version symfony diverge in use of parameterbag->all(KEY)

🇮🇱Israel yakoub

It means that with a class like \Gmagick, the code should not include a use \Gmagick; line.

where does that happen in my code ? i still don't understand .

🇮🇱Israel yakoub

@jayprakash.kushwah thank you for you help .
but i don't think the lines removing the use statements are correct .
can you explain the issue with : Non-namespaced classes/interfaces/traits should not be referenced with use statements

🇮🇱Israel yakoub

have you tested your patch ? it takes more than version value to make it compatible with 10 .

Production build 0.71.5 2024