Skip to content
This repository has been archived by the owner on May 17, 2023. It is now read-only.

Range not found (line 373, file "Code") #18

Open
DrWhodini opened this issue Aug 11, 2014 · 13 comments
Open

Range not found (line 373, file "Code") #18

DrWhodini opened this issue Aug 11, 2014 · 13 comments

Comments

@DrWhodini
Copy link

Firstly - For someone who advertises 'lack of' support, it's great to see you replying to our niggles and problems.
Secondly - I'm loving these scripts, they are great motivational tools that make a difference to how I write.
Lastly - The error - Installation was pretty straightforward, although I needed to totally quite drive and log back in for the stats to work. The Almanac, however repeatedly comes up with this the same error - Range not found (line 373, file "Code")

370 function getNamedRangeCellValue(strRangeName)
371 {
372 var ss = SpreadsheetApp.openById(QS_WRITING);
373 if (ss.getRangeByName(strRangeName) != null)
374 return ss.getRangeByName(strRangeName).getValues()[0];
375 else
376 return null;
377 }

I'm no coder, but I can sometimes get a feel where an issue may lie. Here, however, I'm lost.

Thanks

@OthelloPrime
Copy link

Dr. Whodini (love the name, btw) have you attempted any further troubleshooting on your error? I re-installed the scripts in an attempt to correct an error I was having. My error went away but now I get the exact error you do.

Thanks.

@jamietr1
Copy link
Owner

The most common reason I've seen for this error is that someone goes in an manually changes the value in cell B1 or B2 on the data tab in order to put in their current streak. But you can't overwrite those values because they are calculated and if you overwrite them, you'll break the scripts.

@OthelloPrime
Copy link

I started over without my previous data. (I preserved it in another spreadsheet.) The only number that exists is the goose egg created when I manually ran the WritingStats script earlier.

@OthelloPrime
Copy link

LOL, well I fixed this issue by switching to the Beta track. Now, I get a different error. I'll post the error and any comments there so I don't hijack this thread. Thanks Jamie!

@rustuptwist
Copy link

I had trashed all my scripts files and started over from scratch and I got this same error as well. I as getting both a writingstats and Daily Almanac error each day. Now just this has appeared. When I run the 'run' button in the script it also gives this error. I logged out of my gmail account and logged back in to see if that would change anything. It did not. Any ideas?

@coes123
Copy link

coes123 commented Sep 12, 2014

I am having the exact same problem on a fresh install :( Not sure where to even begin.

@coes123
Copy link

coes123 commented Sep 15, 2014

Guys I found the way to fix this!! All I did was check the Execution Transcript after each error to realize the problem was that the cell names need to be assigned on the Records sheet. Here, I'll explain:

All you have to do is assign names to cells B3, C3, D3 and E3 as follows:

B3: RecordWritingDate
C3: RecordWritingWords
D3: RecordWritingStreak
E3: RecordWritingGoalStreak

Do this and run the getAlmanacText trigger and you are done. Mine is finally working! Good luck.

1

2

@coes123
Copy link

coes123 commented Sep 15, 2014

Ok new issue. Both scripts are running well and sending the email to Evernote BUT DailyAlmanac is not counting the words. I'm getting a "You wrote a total of NaN words." Here's a screenshot. I'll keep poking around.

I already tried test and non test modes.

image

@jamietr1
Copy link
Owner

Question on this: were you using the version of the spreadsheet that I
shared (the one listed in the readme) or was this an earlier version of
your own spreadsheet? I thought I'd included the names in the shared
version, but version of the spreadsheet prior to the shared version might
help explain why the cells are not named. When I have time, I could add
some initialization code that does a kind of validation check on the
spreadsheet and adds anything it finds missing.

Good catch!


Jamie Todd Rubin (@jamietr http://twitter.com/jamietr)
http://www.jamietoddrubin.com
http://www.jamietoddrubin.com/going-paperless/

On Mon, Sep 15, 2014 at 12:34 PM, coes123 notifications@github.com wrote:

Guys I found the way to fix this!! All I did was check the Execution
Transcript after each error to realize the problem was that the cell names
need to be assigned on the Records sheet. Here, I'll explain:

All you have to do is assign names to cells B3, C3, D3 and E3 as follows:

B3: RecordWritingDate
C3: RecordWritingWords
D3: RecordWritingStreak
E3: RecordWritingGoalStreak

Do this and run the getAlmanacText trigger and you are done. Mine is
finally working! Good luck.

[image: 1]
https://cloud.githubusercontent.com/assets/8754843/4274679/d5891ac8-3cf5-11e4-968e-6a901e57d69c.PNG


Reply to this email directly or view it on GitHub
#18 (comment)
.

@coes123
Copy link

coes123 commented Sep 15, 2014

I'm sorry, I can't find that Shared Spreadsheet you mention on any readme :( I'll keep looking. Thank you.

@jamietr1
Copy link
Owner

It should be listed in the readme in the beta-version-2 branch.


Jamie Todd Rubin (@jamietr http://twitter.com/jamietr)
http://www.jamietoddrubin.com
http://www.jamietoddrubin.com/going-paperless/

On Mon, Sep 15, 2014 at 1:02 PM, coes123 notifications@github.com wrote:

I'm sorry, I can't find that Shared Spreadsheet you mention on any readme
:( I'll keep looking. Thank you.


Reply to this email directly or view it on GitHub
#18 (comment)
.

@coes123
Copy link

coes123 commented Sep 15, 2014

Hi,

I found your shared spreadsheet file but I ran into a new error.

Cannot find method formatDate(string,string,string). (line 359, file "Code")

I checked issue #16 and followed the instructions, such as making sure both scripts and the spreadsheet had the same time zone (I am in CDT). I tried EST and CDT to no avail. I also made sure the spreadsheet wasn't in Test Mode and I added the sandbox folders... I am lost now.

I'll keep trying :(

Btw, I changed the timezone in DailyAlmanac in lines 25, 231 and 359.

359 var curDate = Utilities.formatDate(data[i][j], "EDT", "MM/dd/yyyy");

Oh and I also changed the spreadsheet ID on both scripts.

@coes123
Copy link

coes123 commented Sep 15, 2014

What's curious is this. If I switch the DailyAlmanac script to my original spreadsheet the Script runs well but it emails me the NaN (not a number) error. What's also interesting is that my original spreadsheet is in a different time zone setting than the scripts. This, in my opinion, rules out the problem being the time zones. If has something to do with the cells.

I tried removing number formatting on everything too, just in case. I got nothing...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants