Skip to content
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

[Bug: Import] Reimport a renamed tcx file does not change the importfilename in my tourbook anymore #1171

Closed
wolfgang-ch opened this issue Aug 26, 2023 Discussed in #1170 · 2 comments
Milestone

Comments

@wolfgang-ch
Copy link
Collaborator

Discussed in #1170

Originally posted by evdhoek August 24, 2023
In the previous version before 23.8.0, after reimport a renamed tcx file, a helpscreen was shown with 4 options to adapt the new name into the database.
I do not know wheater it is has to do with the recent update, but now I do not see the helpscreen anymore and the importfilename in my tourbook is not changed anymore.
Old name already in MyTourbook: 20230617_1030 f Enschede-TechnBase+ 35.14
Changed into: 20230617_1030 f Enschede, NS-TechnBase+ 35.14
I hope someone can help.

After import I read In Tourlog:
15:41:21 [main - 1] 1 Importing tours...
15:41:21 [Importing tours - 54] 2 [TCX] C:\Users\elevd\OneDrive\Documents.Garmin.Import\20230617_1030 f Enschede, NS-TechnBase+ 35.14.tcx - is empty
15:41:21 [Importing tours - 54] 3 0.531 s ∙ 2,474 # ∙ 17/06/2023, 10:31 ← C:\Users\elevd\OneDrive\Documents.Garmin.Import\20230617_1030 f Enschede, NS-TechnBase+ 35.14.tcx
15:41:21 [Loading imported tours - 55] 4 [Skipped import file] C:\Users\elevd\OneDrive\Documents.Garmin.Import\20230617_1030 f Enschede, NS-TechnBase+ 35.14.tcx in tour 17/06/2023, 10:31
15:41:22 [main - 1] 5 Imported in 1.629 s

@wolfgang-ch wolfgang-ch added this to the 23.next milestone Aug 26, 2023
@wolfgang-ch
Copy link
Collaborator Author

wolfgang-ch commented Aug 26, 2023

When you run MT with the terminal output you will see this exception

java.lang.NullPointerException: Cannot invoke "org.eclipse.swt.widgets.Label.computeSize(int, int)" because "this.imageLabel" is null
	at net.tourbook.common.dialog.MessageDialogWithRadioOptions.createCustomArea(MessageDialogWithRadioOptions.java:112)
	at org.eclipse.jface.dialogs.MessageDialog.createDialogArea(MessageDialog.java:335)
	at org.eclipse.jface.dialogs.IconAndMessageDialog.createDialogAndButtonArea(IconAndMessageDialog.java:228)
	at org.eclipse.jface.dialogs.IconAndMessageDialog.createContents(IconAndMessageDialog.java:207)
	at org.eclipse.jface.window.Window.create(Window.java:431)
	at org.eclipse.jface.dialogs.Dialog.create(Dialog.java:1094)
	at org.eclipse.jface.window.Window.open(Window.java:788)
	at org.eclipse.jface.dialogs.MessageDialog.open(MessageDialog.java:609)
	at net.tourbook.importdata.RawDataManager.lambda$4(RawDataManager.java:1995)
	at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:40)
	at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:132)

In Eclipse 4.25, some icons were removed in standard dialogs https://eclipse.dev/eclipse/news/4.25/platform.php#less-icons-in-user-dialogs and the code in MT, which is copied/modified from Eclipse, is causing this exception because MT before 23.8 was using Eclipse 4.24 which had icons in this dialog

This error was also visible in the error log view

image

wolfgang-ch added a commit that referenced this issue Aug 26, 2023
@wolfgang-ch
Copy link
Collaborator Author

Fixed in bc79a05

wolfgang-ch added a commit that referenced this issue Aug 26, 2023
@wolfgang-ch wolfgang-ch changed the title [Import] Reimport a renamed tcx file does not change the importfilename in my tourbook anymore [Bug: Import] Reimport a renamed tcx file does not change the importfilename in my tourbook anymore Aug 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant