Single Cell endRowIndex

[Max-Makhrov]

Dear Tanaike, I use this script a lot in my work. thank you a ton!

Here's the issue I found:

Sample Code

function testApp() {
  var a1Notation = 'K1';
  var sheetId = 500;
  console.log(UtlApp.convA1NotationToGridRange(a1Notation, sheetId));
}

Output

{ sheetId: 500,
  startRowIndex: 0,
  endRowIndex: 1,
  startColumnIndex: 10,
  endColumnIndex: 1 }

Expected output

{ sheetId: 500,
  startRowIndex: 0,
  endRowIndex: 1,
  startColumnIndex: 10,
  endColumnIndex: 11 }

Steps to reproduce

1. Install the library
2. run sample code

Proposed solution

My guess is that it is a logical error in this line:

endColumnIndex: end ? end.col + 1 : 1,

I suggest to change it like this:

endColumnIndex: end ? end.col + 1 :start.col+1

If I did not understand the logics of grid creation, I'm sorry.

I also suggest to create a type for a grid like so for autocompletion purposes:

/**
 * @typedef {Object} RangeGrid
 * @property {Number} startRowIndex
 * @property {Number} endRowIndex
 * @property {Number} startColumnIndex
 * @property {Number} endColumnIndex
 */

 *
 * ...
 * @returns {RangeGrid} Gridrange.
 */
function convA1NotationToGridRange_(a1Notation, sheetName) {

[tanaikech]

Thank you for your support. And, I apologize for my late response. Now, I noticed your comment.

From your comment, I confirmed it. So, I updated this library as v1.0.1. Could you please confirm it?

Thank you so much.

[Max-Makhrov]

Thanks, @tanaikech !

Your library and no-apps-script approach makes scripts work much faster, and also help developing Front-End part. conv-functions became my best friends :)

[tanaikech]

Thank you for replying. Yes. This repository publishes the functions as both a library and the standalone functions in order to match the user's situation. I'm glad that those are useful for your situation. I would like to grow this repository by adding various functions.