The recordset's repr() changes and is misleading

[yajo]

Look at how self looks in VSCode without the plugin:

With it:

As said in Tecnativa/doodba#333 (comment), the fact that self becomes [1, 2, 3] instead of stock.warehouse(1, 2, 3) is misleading, as it makes the variable look like it's a list instead of a recordset.

I wonder if that can be left as it was, without impacting the other niceties of this plugin.

[ntvsp]

a ngọc làm plugin cho vscode rồi à

[yajo]

Where is it?

[cuongnmtm]

@yajo: tuanvuvd said that the author finally created the plugin for vscode. It's like a rhetorical question

[gdgellatly]

@cuongnmtm Yes we understand it works for VSCode, it was more why the repr() output disappears in VSCode. In the first screenshot you clearly see the repr() representation (without this plugin), in the screenshot in the README.md for this plugin you clearly see the repr() representation with the name, in the screenshot above, in VSCode there is no repr(). i.e. 2: {stock.warehouse: 3} [3] Chicago or similar

[cuongnmtm]

@gdgellatly I tried to translate for Yajo

BTW, I like the way @trinhanhngoc adjusted repr() to show the record name. It's especially useful in case there are relationship fields inside

[cuongnmtm]

@yajo @gdgellatly My pull request was merged. Could you please check the result? I think this issue is ok to close

[yajo]

Yes, pretty cool, thanks!