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

v14.1 #4649

Merged
merged 885 commits into from
May 27, 2024
Merged

v14.1 #4649

merged 885 commits into from
May 27, 2024

Conversation

SurfaceS
Copy link
Contributor

@SurfaceS SurfaceS commented May 14, 2024

Description of code changes

Implements latest Jetty (v12.0.8).
Implements latest JUPnP (v3.0.1).
Removed the JDK HTTP servlet server (personal implementation).
Stop using the deprecated tool 'Create React App' (no longer being maintained)
Implements Vite as replacement.

The webgui / webplayer are now served by Jetty (servlet).

Goal of the version

Remove multiple engine maintenance.
Remove Netty without stress. #4058

To test

We should focus on the webgui / webplayer tests first, as it is now served by Jetty servlets.
It will be easy to test as it is not changing settings / database or so that will break if we revert to v14 build.

Netty is kept as the default engine, letting us time to fix Jetty implementation (by changing the default engine in the GUI settings).
Once Jetty implementation is fully tested, we can remove all Netty related things (engine, deps, and so on).

Checklist

  • Any relevant issues are referenced
  • Any relevant Knowledge Base articles are written/modified
  • Any relevant tests have been written/modified

SurfaceS and others added 30 commits February 6, 2024 10:59
# Conflicts:
#	react-client/package.json
#	react-client/yarn.lock
V14 fix empty upnp results
updated the browsers logo to the 2024 one.
V14 - added ability to use svg files for renderer images.
…ted.

scanner : don't try to browse parent shared root (outside share).
add a link to the new maintained documentation.
fix html/css.
centralize helpPage things under it's own class.
i18n get to pass value if no translation exists
# Conflicts:
#	react-client/yarn.lock
fix MediaScanner ignored folders list
# Conflicts:
#	react-client/package.json
#	react-client/yarn.lock
V14 - added option to disable pool limits (globally or individually).
fix some labels where not shown
allow showAdvancedOptions to be dynamic (no restart).
allow language change to be dynamic (partially no restart).

Reflect more the react gui settings :
added DisableAllTranscoding
moved some settings according to the react gui.
recreate playlist folder if not known to renderer
# Conflicts:
#	react-client/package.json
#	react-client/yarn.lock
# Conflicts:
#	src/main/java/net/pms/store/MediaStore.java
@SurfaceS
Copy link
Contributor Author

@SurfaceS ok cool, are you happy for it to be merged? I haven't found any problems with it

@SubJunk yes.

next step (v14.2) will be to use the jetty media server implementation as default, then we can remove completely netty.

@SurfaceS
Copy link
Contributor Author

@SubJunk I made a cleanup in the changelog, then it will be easier to merge, after you build the 14.0.1 to fix 14.0.0 regressions.

I think we should build the 14.0.1 before releasing the 14.1.0, as you may not want to give the 14.0.0, that users will create issues (regression) already solved on 14.0.1.

@SubJunk
Copy link
Member

SubJunk commented May 24, 2024

@SurfaceS ok, I will release 14.0.1 today

@SubJunk
Copy link
Member

SubJunk commented May 26, 2024

I got this error on this branch, not sure whether it is related to it

08:34:07 INFO  Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException: Cannot invoke "java.awt.Window.pack()" because the return value of "javax.swing.SwingUtilities.getWindowAncestor(java.awt.Component)" is null
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererFrame.buildEditBar(RendererFrame.java:131)
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererFrame.buildPanel(RendererFrame.java:99)
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererFrame.update(RendererFrame.java:78)
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererPanel.lambda$updateRenderer$1(RendererPanel.java:200)
08:34:07 INFO  	at java.desktop/java.awt.event.InvocationEvent.dispatch(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventQueue.dispatchEventImpl(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventQueue$4.run(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventQueue$4.run(Unknown Source)
08:34:07 INFO  	at java.base/java.security.AccessController.doPrivileged(Unknown Source)
08:34:07 INFO  	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventQueue.dispatchEvent(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventDispatchThread.pumpEvents(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventDispatchThread.pumpEvents(Unknown Source)
08:34:07 INFO  	at java.desktop/java.awt.EventDispatchThread.run(Unknown Source)

@SurfaceS
Copy link
Contributor Author

I got this error on this branch, not sure whether it is related to it

08:34:07 INFO  Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException: Cannot invoke "java.awt.Window.pack()" because the return value of "javax.swing.SwingUtilities.getWindowAncestor(java.awt.Component)" is null
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererFrame.buildEditBar(RendererFrame.java:131)
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererFrame.buildPanel(RendererFrame.java:99)
08:34:07 INFO  	at net.pms.swing.gui.tabs.status.RendererFrame.update(RendererFrame.java:78)

It is not related to this branch, but it is fixed now

@SubJunk SubJunk merged commit 0ff265f into main May 27, 2024
19 checks passed
@SubJunk
Copy link
Member

SubJunk commented May 27, 2024

This is working really well for me. The performance improvement while browsing is very noticeable on my TV

sf666 added a commit to ik666/UniversalMediaServer that referenced this pull request May 27, 2024
* main_UMS:
  v14.1 (UniversalMediaServer#4649)
  main fix NullPointerException on RendererFrame.buildEditBar (UniversalMediaServer#4701)

# Conflicts:
#	pom.xml
#	src/main/java/net/pms/network/mediaserver/javahttpserver/MediaServerHandler.java
@SurfaceS SurfaceS deleted the v14.1 branch May 28, 2024 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants