Fix null item handling in toNotch (#103)

extremeheat · web-flow · commit 0fd5781c1c53 · 2023-07-22T16:42:17.000+02:00
fix error on undefined item https://github.com/PrismarineJS/prismarine-windows/actions/runs/5630856679/job/15257229603#step:5:30
diff --git a/index.js b/index.js
@@ -68,7 +68,7 @@ function loader (registryOrVersion) {
     }
 
     static toNotch (item, serverAuthoritative = true) {
-      const hasNBT = item.nbt && Object.keys(item.nbt.value).length > 0
+      const hasNBT = item && item.nbt && Object.keys(item.nbt.value).length > 0
 
       if (registry.type === 'pc') {
         if (registry.supportFeature('itemSerializationAllowsPresent')) {

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ function loader (registryOrVersion) {`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`static toNotch (item, serverAuthoritative = true) {`
`71`		`- const hasNBT = item.nbt && Object.keys(item.nbt.value).length > 0`
	`71`	`+ const hasNBT = item && item.nbt && Object.keys(item.nbt.value).length > 0`
`72`	`72`
`73`	`73`	`if (registry.type === 'pc') {`
`74`	`74`	`if (registry.supportFeature('itemSerializationAllowsPresent')) {`