Hidden columns not expandable [2.x]

Same patch than #3093730 🐛 Hidden columns not expandable [1.x] Needs review . Could be applied successfully for the 2.0.0 version. Not for the 2.x-dev one since the file datatables.js has changed. Here this patch for that version.

#2

Nothing new here? @Anybody, not convinced? May be, to move forward,
- does the suggested code solves the problem? If not what are its functional problems?
- does the suggested code is better than the current one?
- if this code is not good, which code rather?

More arguments. Found in Drupal core 10.1 or 9.5,
theme.prototype: 0 occurrences
Drupal.theme.: >50 occurrences
In contrib/,
theme.prototype: 1 occurrence: datatables/js/datatables.js:139
Drupal.theme.: >20 occurrences

If someone else can confirm and explain this change is correct, we can set this RTBC this of course.

-   Drupal.theme.prototype.datatablesExpandableRow = function (datatable, row) {
+   Drupal.theme.datatablesExpandableRow = function (datatable, row) {

@Chris64 could you please prepare #4 as MR for further review?

MR ready.

General RTBC for MR!13 still this is a case, where tests would help a lot to ensure we don't break any (edge)cases. The maintainer should decide. At least I'd merge this last and keep it in dev for a while, before tagging a release.

Again, I think we are not on the road breaking things badly at the moment and in times when things move forward slowly I recommend to go forward before cutting nervs of contributors with tests. I would stand on tests more if this project would have more critical impact in the center of a Drupal installation but in this case here and especially because at the moment it is a hand full of contributors pushing things forward here only, I will take the heard and courage to push things and wait until some things pop up ;-)

Again thanks @ all for all the hard work.

Merged to latest dev and prepared for queue into 2.0.1 (Drupal 10 compatible) release this month.

Automatically closed - issue fixed for 2 weeks with no activity.