Problem with 'Edit Songs Metadata' feature

[rush_fan]

The first time I use it, almost always to replace artwork, everything’s fine. I browse to the new album art, replace artwork, and save. It saves and the window closes.

However, any subsequent attempt to go into ‘Edit Songs Metadata’ fails. Songkong appears to freeze and no editing window comes up (at least none that I can see). It’s as if the edit window has spawned somewhere but it’s not visible. The only way to proceed from here is to kill songkong and re-run it.

songkong version: 9.6 Whirlpool (Pro)
OS version: Pop!_OS 22.04 LTS x86_64 (linux 6.5.4-76060504-generic)

I can re-create the problem at will if there’s anything you’d like me to check.

[paultaylor]

Hmm, just tried and it works for me so please recreate issue then immediately after run Create Support Files so I can check logs for errors

[paultaylor]

Okay so there is an error so I have raised issue https://jthink.atlassian.net/browse/SONGKONG-2513

java.lang.IllegalArgumentException: no Action for action key: selectNextColumnCell
	at com.jthink.songkong.ui.swing.WrappedAction.setActionForKey(WrappedAction.java:86)
	at com.jthink.songkong.ui.swing.WrappedAction.<init>(WrappedAction.java:39)
	at com.jthink.songkong.ui.swing.EditableCellFocusAction.<init>(EditableCellFocusAction.java:16)
	at com.jthink.songkong.ui.swing.SongKongTable.setFocus(SongKongTable.java:178)
	at com.jthink.songkong.ui.startdialog.editsongs.ArtworkTab.<init>(ArtworkTab.java:144)
	at com.jthink.songkong.ui.startdialog.editsongs.StartEditSongsDialog.setupDialog(StartEditSongsDialog.java:100)
	at com.jthink.songkong.ui.startdialog.editsongs.StartEditSongsDialog.<init>(StartEditSongsDialog.java:53)
	at com.jthink.songkong.action.EditSongsAction.startTask(EditSongsAction.java:46)
	at com.jthink.songkong.action.SongKongPrimaryTaskAction.actionPerformed(SongKongPrimaryTaskAction.java:130)
	at java.desktop/javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1972)
	at java.desktop/javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2313)
	at java.desktop/javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:405)
	at java.desktop/javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:262)
	at java.desktop/javax.swing.AbstractButton.doClick(AbstractButton.java:374)
	at java.desktop/javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1028)
	at java.desktop/javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1072)
	at java.desktop/java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:297)
	at java.desktop/java.awt.Component.processMouseEvent(Component.java:6626)
	at java.desktop/javax.swing.JComponent.processMouseEvent(JComponent.java:3389)
	at java.desktop/java.awt.Component.processEvent(Component.java:6391)
	at java.desktop/java.awt.Container.processEvent(Container.java:2266)
	at java.desktop/java.awt.Component.dispatchEventImpl(Component.java:5001)
	at java.desktop/java.awt.Container.dispatchEventImpl(Container.java:2324)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4833)
	at java.desktop/java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4948)
	at java.desktop/java.awt.LightweightDispatcher.processMouseEvent(Container.java:4575)
	at java.desktop/java.awt.LightweightDispatcher.dispatchEvent(Container.java:4516)
	at java.desktop/java.awt.Container.dispatchEventImpl(Container.java:2310)
	at java.desktop/java.awt.Window.dispatchEventImpl(Window.java:2780)
	at java.desktop/java.awt.Component.dispatchEvent(Component.java:4833)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:773)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:722)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:716)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:97)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:746)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:744)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:743)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

But don’t understand the cause of the issue currently, does it fail if you go into Edit Songs Metadata but do not replace artwork?

[rush_fan]

No. If I change the album name, for example, it’s fine. Same is true if I simply start the metadata editor and cancel out. So it might only be when you replace artwork.

[paultaylor]

@rush_fan

So it might only be when you replace artwork.

I did try that and it still worked for me but it maybe related to the way you are doing it somehow. Okay sure I will work it out but might take a few days.

[paultaylor]

Fixed for next release.