Bug: Improperly storing MediaStreams2

[joshua4 12]

This is the command emby runs on videos and its output

> ffprobe.exe -v error -select_streams v:0 -show_entries stream=width,height,duration,bit_rate -of default=noprint_wrappers=1 "C:\video.mkv"
width=1920
height=1080
duration=N/A
bit_rate=N/A

These values are stored in the database as

Height | Width
1920   | 1080

Oops.

[Carlo 4561]

Can you give us more info on the version of Emby you are seeing this on?

[joshua4 12]

4.5.4.0

[Carlo 4561]

Thanks for reporting.

[Carlo 4561]

I just ran a SQL statement on my Database which has been through untold versions and I don't see anything like this.

What do you get if you run this SQL?

select Height, Width from MediaItems where Height > Width

 

 

[joshua4 12]

MediaStreams2

[Carlo 4561]

What was your source of media?  What type of media?

[joshua4 12]

mkv

[Carlo 4561]

I see quite a few rows that have this wrong.

@Luke

[Luke 42080]

Hi, yes it is an old defect. The naming of the columns was inadvertently flipped, but the code handles so it is essentially just a typo.

[joshua4 12]

37 minutes ago, Luke said:

Hi, yes it is an old defect. The naming of the columns was inadvertently flipped, but the code handles so it is essentially just a typo.

What's the diff between MediaItems Width/Height and MediaStreams2 Width/Height?

[Luke 42080]

36 minutes ago, joshua4 said:

What's the diff between MediaItems Width/Height and MediaStreams2 Width/Height?

For videos, none. It's duplicated there because some things like photos need width/height but don't use mediastreams.