-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Diff URLs truncated #26
Comments
Well that’s odd. It looks to do with the version string. See the missing “a4” at the end of “alpha4” for fast_404? I’m on mobile at the moment but lll take a look as soon as I can.
Thanks for reporting it!
… On May 7, 2020, at 6:50 PM, Chris Burgess ***@***.***> wrote:
Just testing this out (thanks!) and noticed that some of the URLs provided for compare are truncated. Eg:
| drupal/entity_reference_revisions | 1.7.0 | 1.8.0 | https://git.drupalcode.org/project/entity_reference_revisions/compare/8.x-1.7...8.x-1.8 |
| drupal/fast_404 | 1.0.0-alpha4 | 1.0.0-alpha5 | https://git.drupalcode.org/project/fast_404/compare/8.x-1.0.0-alph...8.x-1.0.0-alph |
| drupal/page_manager | 4.0.0-beta4 | 4.0.0-beta5 | https://git.drupalcode.org/project/page_manager/compare/8.x-4.0.0-bet...8.x-4.0.0-bet |
| drupal/views_data_export | 1.0.0-beta4 | 1.0.0-rc1 | https://git.drupalcode.org/project/views_data_export/compare/8.x-1.0.0-bet...8.x-1.0.0-r |
(I've intentionally excerpted to show a single correct URL and some that are not working links.)
tput cols returns 236; these are not shortened due to limitations of the console AFAICT. See screenshot below.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
The cause isn't obvious to me either. It would be remiss to not share this E_NOTICE output, even though I don't think it's related.
(that appears twice in the output, but not as many times as there are truncated URLs, therefore I doubt it's the cause ... but it is coming from If I can't track the problem down I'll attach a sample composer.json & lock to demo 😁 |
The truncation is strange. The second line of your example is truncated at 14 characters and the next two are at 13. The only place in the code that manipulates the version received from the lock files is 75 and that truncates to 7 places for git hashes. Can you provide the source files? Also, are you using a current version? Line 212 is
The |
I think this is corrected with the changes to drupalcode handling in 1.6.0. Please reopen if it is not. |
Just testing this out (thanks!) and noticed that some of the URLs provided for compare are truncated. Eg:
(I've intentionally excerpted to show a single correct URL and some that are not working links.)
These output URLs do not provide comparison views correctly, while most do:
In worst case this might result in deceptive output (eg diff vs -beta1 links to -beta, or diff vs -beta11 links to -beta1).
tput cols
returns 236; these are not shortened due to limitations of the console AFAICT. See screenshot below.The text was updated successfully, but these errors were encountered: