Bugs with reading data from sheets when there's no data to return (empty sheets/range) (built-in spreadsheet component)

[Patryk_Gesiewicz]

ReadSheet, ReadWithPartialFilter, ReadWithExactFilter methods - while trying to read an empty sheet it throws an error: “RetrieveSheet: Unknown Error: Attempt to invoke interface method ‘int java.util.List.size()’ on a null object”.
Someone forgot to check if there is any data.

ReadColumn, ReadRow, ReadRange just throw an error “No data found”.

ReadCell just returns empty result.

All methods should return empty data (lists) if there are no data as this is not an error. The way it works no does not allow for checking if we have an empty sheet or cells (ReadRange) or even to search a sheet for specific data (read with filters).

[ewpatton]

You are correct. I've filed an issue so we can track this.

Spreadsheet component throws errors on empty read

[yashsrv]

@ewpatton ReadCell returns empty string, I think that’s the expected result?

PS: A fix is here

Fix SpreadSheet Read Blocks (#4094)

[ewpatton]

I think it's correct that ReadCell should provide an empty text if there is nothing there. The only exception would be if they attempt to read a cell outside the bounds of the sheet, in which case that should raise an error.

[yashsrv]

I tried to read cell A1001 that resulted in an error, so this block is working fine.

[ABG]

See

Begin testing Release nb207

We have updated ai2-test.appinventor.mit.edu as we begin the testing of release nb207. Changes from nb206 Upgrade the blocks system to Blockly version 13. Fixed scrolling in multiline, readonly Textbox. Bugfixes to Text to Speech. Fixed a value rounding bug in slider. Resize the warning toggle to fit its current label, invoked when the toggle text is updated (Show Warning/Hide Warning) and after dom creation. Change “Import Extension” from a link to a button. Add a new "titlecase" operator f…

Please test.

[Patryk_Gesiewicz]

Wow, this was really quick.

It works perfectly. All methods now return empty lists without any errors and throw errors only if there’s an actual error (wrong sheet name, range, cell/column/row reference).

[yashsrv]

It feels so good after seeing your response