Skip to content

Commit

Permalink
fixed 'uniques' list selector support for nested atomic lists
Browse files Browse the repository at this point in the history
  • Loading branch information
jmoenig committed Nov 21, 2023
1 parent 5641c14 commit 740c4f0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
3 changes: 2 additions & 1 deletion HISTORY.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,12 @@

## in development:
* **Notable Fixes:**
* fixed 'distribution' list selector support for nested atomic lists
* fixed 'distribution' and 'unique' list selectors support for nested atomic lists

2023-11-21
* new dev version for v9.0.14
* threads: fixed 'distribution' list selector support for nested atomic lists
* threads: fixed 'uniques' list selector support for nested atomic lists

## 9.0.13:
* **Notable Fixes:**
Expand Down
13 changes: 4 additions & 9 deletions src/threads.js
Original file line number Diff line number Diff line change
Expand Up @@ -2278,15 +2278,10 @@ Process.prototype.reportListAttribute = function (choice, list) {
case 'uniques':
this.assertType(list, 'list');
if (list.canBeCSV()) {
if (Process.prototype.isCaseInsensitive) {
return list.map(row => row instanceof List ?
row.map(cell =>
cell.toString().toLowerCase()
)
: row.toString().toLowerCase()
).distribution().columns().at(1);
}
return list.distribution().columns().at(1);
return this.reportListAttribute(
'distribution',
list
).columns().at(1);
}
return this.reportUniqueValues(list);
case 'distribution':
Expand Down

0 comments on commit 740c4f0

Please sign in to comment.