No, and it is in fact broken. Firstly, you never register this type, so it will never work correctly. Secondly, you must not create registry entries in static initializers. You did it correctly above:
Use this instead. However this is in the wrong class, as particle types (as all registry entries) must be registered on both sides (client and server). As such doing this in a client-only class will never work right.
Every time you post you post in the wrong subforum and I have to move your thread. Please take the hint.
This is broken. You must not return null from this method. Use NetworkHooks.getEntitySpawningPacket.
You must also only spawn your entity and start riding it on the server.