Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Is it normal that until I installed this module, not appeared any language update?
Thanks!
Comments
Comment #1
rodrigoaguileraI'm curious about this too
Comment #2
cilefen CreditAttribution: cilefen commentedComment #3
penyaskitoThis is on purpose. The project data is based on the project list supplied by the Update module, so it degrades gracefully to not checking for translations. Maybe this should be more explicitly remarked on the page?
Pinging Gábor about it.
Comment #4
Gábor HojtsyYes, this is by design. The UI should inform people about it. We need the version information and update status is the only module that can tell us about that.
Comment #5
Gábor HojtsyBTW the idea is update manager can update the project information that locale uses, but the project information is cached and available from the last time the update module was still enabled.
Comment #6
Gábor HojtsyApparently nobody is working on this one, so removing from sprint.
Comment #7
Gábor HojtsyFix incorrect title.
Comment #8
Jose Reyero CreditAttribution: Jose Reyero as a volunteer commentedThere's nothing here that degrades gracefully, unless you can call "degrade gracefully" to "not working".
I suggest:
1. Removing all UI mentions to translation updates, included the Reports/Translation updates pages completely when Update Manager module is not enabled. Really if it's not working I'd prefer not seeing it.
2. Adding to Update Manager module description: "Checks for available updates for ... and Translations"
This way, the functionality may look like coming from Update Manager module thought it is built into the Locale module. No one will wonder why they don't get translation updates if they don't have Translation (Locale) enabled.
This may look ugly but at least it won't confuse users that much.
Comment #9
bsztreha CreditAttribution: bsztreha commentedI've created the patch. Is this the right way?
Comment #10
penyaskito@bsztreha Thanks for working on this!
For the text is perfect I think, but for the translations form I think Jose is suggesting to remove the page. We could do that by using _module_dependencies in the routing YAML file providing that form (See docs at https://www.drupal.org/node/2092643).
Comment #11
Gábor HojtsyIn that case, let's update the issue title and summary.
Comment #12
bsztreha CreditAttribution: bsztreha commentedI've attached the new patch, that contains the new hook, because locale.links.menu.yml was break Drupal
Comment #13
bsztreha CreditAttribution: bsztreha commentedComment #14
Gábor HojtsyHow was this one not enough?
Comment #15
Gábor HojtsyMissing "Implements ..." docblock.
Comment #16
bsztreha CreditAttribution: bsztreha commented#14
Because of the _module_dependencies, if update module does not exists, locale.links.menu.yml want to create menu, but route does not exists, and you get an error page on clear cache
Comment #17
Gábor HojtsyHa, good catch!
Comment #19
bsztreha CreditAttribution: bsztreha commentedI hope i've repaired one of fail, and i marked the places where we use Url with this path.
Comment #20
Gábor HojtsyWhere is this function even invoked?
Comment #21
bsztreha CreditAttribution: bsztreha commentedHonestly i dont know, i think its not in use.
In my opinion this way is not good, because is anybody create a link like:
then Drupal page will be failed.
I dont how to solve this kind of url call, in this case the module exists check is overhead.
Comment #22
Gábor HojtsyI don't think that is an overhead. The same happens for routes of whatever module that may or may not be enabled.
Comment #23
bsztreha CreditAttribution: bsztreha commentedI have to separated the hook_help description, where is linked the locale.translate_status Url.
Is this what you thinking about?
Comment #24
Sutharsan CreditAttribution: Sutharsan as a volunteer commentedWe should not fix it here. We should fix locale_translation_build_projects(). The fallback solution there is quick and dirty. A better solution is already available in the l10n_update module.
It is not the update module that checks for translation updates. It is (currently) a requirement.
Comment #25
Sutharsan CreditAttribution: Sutharsan as a volunteer commented@bsztreha Please add an interdiff if you make a new patch.
Comment #26
Sutharsan CreditAttribution: Sutharsan commentedI've created #2550137: Improve interface translation fallback if Update module is disabled to fix the problem at the right end. We may still need some additional user notification, but only to tell the admin that better results may be achieved in when dev module translations are missing or not being updated.
Comment #27
Gábor HojtsyWell, its not just dev modules but falling back on older releases too, no?
Comment #28
Sutharsan CreditAttribution: Sutharsan commentedUpdate module is only used to determine a fallback in case of for dev releases. Since #2550137 only catches dev released after their first release, the content of the status page may be off. We still need some explanation there, especially in this stage of development with a lot of dev releases. I suggest to put a warning on the page like "The translation status of this page may be incomplete. Enable Update module to get accurate information."
Comment #29
bsztreha CreditAttribution: bsztreha commentedI mentioned that at #9, see patch
Comment #30
Gábor HojtsyNote that after #2113957: Build server side version fallback system for translations, we could even explore not depend on update status at all even for the fallback. We can either repurpose this issue or open a new one if people think this is a good idea :) Although the client side has more info than the server redirects using update status' data, it is only slightly different and may work just as well with the server side info and be simpler.
Comment #31
Jose Reyero CreditAttribution: Jose Reyero as a volunteer commented+1 to get rid of that dependency on update status.
The issue is already there, I think this is the one #2113957: Build server side version fallback system for translations
So I'd ask for the patch here to be put 'on hold' for a few days and I'm giving it a try, will follow up on the other issue. I think we all want to get rid of update manager on our production servers while still being able to use localization update.
That server side fallback is great news, btw.
Comment #32
Gábor HojtsyI don't necessarily agree people will want to have localization update on production sites, it depends on the stability of the strings I guess coming from your translation team. If the team like changing translations of important concepts overnight (eg. user, node, theme, etc), that may be a problem :D Localization update can in fact do more harm if you are concerned about the stability of your strings than update status, given that update status merely informs you update things and does not change anything.
That said, I fully supporting making the two independent if we can get reasonably the same feature set, so people have the option to configure their site however they want.
Comment #33
Gábor Hojtsy#1832946: Runtime translation download fallback works different from installer translation download fallback landing obsoletes this issue. Sorry @bsztreha, hope this does not discourage you from contributing. It happens that we have different approaches and some win over others.
Comment #34
bsztreha CreditAttribution: bsztreha commentedIt's no problem :)
I hope i will help in another issue soon