Thread Tools Display Modes
08-05-15, 11:31 PM   #1
Kreelor
A Firelord
 
Kreelor's Avatar
Join Date: Feb 2008
Posts: 495
ERROR..\AddOns\Carbonite.Warehouse\NxWarehouse.lua line 3434:

Error upon starting a new character. It says "...NxWarehouse.lua line 3434:" but, the error happened immediately upon starting the new char without ever having the chance to visit a bank.

-- Using latest DEV version
-- Mists of Pandaria - NOT WoD.
Horde/Mage - Level 1
========================
Attached Files
File Type: lua Error-Carbonite-Starting-New-Char.lua (6.5 KB, 431 views)
  Reply With Quote
08-09-15, 03:22 AM   #2
gOOvER
A Theradrim Guardian
 
gOOvER's Avatar
AddOn Author - Click to view addons
Join Date: Apr 2008
Posts: 62
Line 3434 is a Empty Line..... So are you using really the latest Version?
  Reply With Quote
08-09-15, 05:14 AM   #3
ircdirk
A Molten Giant
 
ircdirk's Avatar
AddOn Author - Click to view addons
Join Date: Nov 2014
Posts: 823
Originally Posted by Kreelor View Post
-- Mists of Pandaria - NOT WoD.
On private server? I dont know if older versions of WoW are supported...
__________________
Carbonite and Carbonite Classic Developer
  Reply With Quote
08-09-15, 05:48 AM   #4
jeffy162
A Pyroguard Emberseer
 
jeffy162's Avatar
AddOn Author - Click to view addons
Join Date: May 2009
Posts: 2,364
@ircdirk: That just means (to him, anyway) that he hasn't bought the WoD expansion package yet.
__________________
Ahhhh, the vagueries of the aging mind. Wait.... What was I saying?


Carbonite <----- GitHub main module (Maps ONLY) download link. The other modules are also available on GitHub.
Carbonite-CLASSIC<----- GitHub link to Carbonite Classic. Thanks to ircdirk for this!
  Reply With Quote
08-09-15, 06:39 AM   #5
ircdirk
A Molten Giant
 
ircdirk's Avatar
AddOn Author - Click to view addons
Join Date: Nov 2014
Posts: 823
Originally Posted by jeffy162 View Post
@ircdirk: That just means (to him, anyway) that he hasn't bought the WoD expansion package yet.
U dont know that, u are assuming it Kreelor should write which version of WoW he usues in numbers not expansions.
__________________
Carbonite and Carbonite Classic Developer
  Reply With Quote
08-09-15, 09:54 AM   #6
jeffy162
A Pyroguard Emberseer
 
jeffy162's Avatar
AddOn Author - Click to view addons
Join Date: May 2009
Posts: 2,364
Originally Posted by ircdirk View Post
U dont know that, u are assuming it Kreelor should write which version of WoW he usues in numbers not expansions.
Yes, I do. He has said as much.

@gOOver: If you read his post, he is using the latest DEV version which, if I remember correctly, is the release version with some minor updates.
__________________
Ahhhh, the vagueries of the aging mind. Wait.... What was I saying?


Carbonite <----- GitHub main module (Maps ONLY) download link. The other modules are also available on GitHub.
Carbonite-CLASSIC<----- GitHub link to Carbonite Classic. Thanks to ircdirk for this!

Last edited by jeffy162 : 08-09-15 at 09:59 AM.
  Reply With Quote
08-10-15, 06:48 AM   #7
Kreelor
A Firelord
 
Kreelor's Avatar
Join Date: Feb 2008
Posts: 495
Originally Posted by gOOvER View Post
Line 3434 is a Empty Line..... So are you using really the latest Version?
Yes. I looked at. I have no idea what you looked at.
Attached Thumbnails
Click image for larger version

Name:	Line_3434_is_not_empty.jpg
Views:	411
Size:	498.6 KB
ID:	8616  
  Reply With Quote
08-10-15, 08:20 AM   #8
Thrumbar
A Chromatic Dragonspawn
Join Date: Jan 2005
Posts: 157
Originally Posted by Kreelor View Post
Yes. I looked at. I have no idea what you looked at.
Current versions of Release, Git Master and Git DEV are empty at that line. Are you sure that you have a current version of that.

Current Dev as of today last update 2 months ago (warehouse module).
It appears atha there is a 5 line difference between your copy and current Dev (same area as your image)..... Please make sure you have the correct version by downloading again but clean off all previous downloaded version from you download folder.

https://onedrive.live.com/redir?resi...nt=photo%2cpng

With regard
Thrumbar

Edit 1: It appears your version of Warhouse.lua is from before Rythal commit as of June 2 (commit 657d247d44005233342d693b1c50d322157ad80c)) that added 5 lines to the Warehouse.lua.

Edit 2: Note that this does not imply that there is no error in relation to the line 3434 of Kreelor's OP only the reason others see the said line in 3439.

Last edited by Thrumbar : 08-10-15 at 11:07 AM.
  Reply With Quote
08-10-15, 04:57 PM   #9
Rythal
Featured Artist
Featured
Join Date: Aug 2012
Posts: 1,458
I merged my local version of dev into live, I did not merge my local dev into dev... it's very possible there is code differences.

I've been without a computer since the first week of July so I can't check local versions, but it's possible it sync'd dev before doing the live version.

Last edited by Rythal : 08-10-15 at 05:01 PM.
  Reply With Quote
08-11-15, 07:35 AM   #10
Kreelor
A Firelord
 
Kreelor's Avatar
Join Date: Feb 2008
Posts: 495
Thanks everybody. I didn't realize this would get so confusing. I was merely trying to help by posting an error that happened to me.

I'm not sure that I understand exactly 'which' version to download and install; DEV or Master, (since nobody told me specifically which one to use). I went ahead and deleted everything related to Carbonite in my Downloads folder and I also deleted all Carbonite folders from my C:\Program Files (x86)\World of Warcraft\Interface\AddOns folder. I'm going to download and install what appears to me as the most recent GitHub files for Carbonite-related modules and, that one is the current DEV version.

I think it's too bad that the GitHub does not use actual dates-times instead of rounding things by months. Both DEV and Master show the same "2 months ago" for many of the files. That doesn't really tell me which version is the latest version in either of them.

The attached image here explains why I guessed that it is probably the one to use. If I guessed incorrectly, I'll appreciate someone telling me I did it wrong.
Attached Thumbnails
Click image for larger version

Name:	DEV-versus-MASTER-version-dates.jpg
Views:	527
Size:	402.3 KB
ID:	8617  
  Reply With Quote
08-11-15, 09:47 AM   #11
ircdirk
A Molten Giant
 
ircdirk's Avatar
AddOn Author - Click to view addons
Join Date: Nov 2014
Posts: 823
Originally Posted by Kreelor View Post
Thanks everybody. I didn't realize this would get so confusing. I was merely trying to help by posting an error that happened to me.

I'm not sure that I understand exactly 'which' version to download and install; DEV or Master, (since nobody told me specifically which one to use). I went ahead and deleted everything related to Carbonite in my Downloads folder and I also deleted all Carbonite folders from my C:\Program Files (x86)\World of Warcraft\Interface\AddOns folder. I'm going to download and install what appears to me as the most recent GitHub files for Carbonite-related modules and, that one is the current DEV version.

I think it's too bad that the GitHub does not use actual dates-times instead of rounding things by months. Both DEV and Master show the same "2 months ago" for many of the files. That doesn't really tell me which version is the latest version in either of them.

The attached image here explains why I guessed that it is probably the one to use. If I guessed incorrectly, I'll appreciate someone telling me I did it wrong.
U can see changes here: https://github.com/Rythal/Carbonite/compare/DEV, there u can see those 3 commits that DEV is AHEAD of MASTER. Not many changes but there are some
__________________
Carbonite and Carbonite Classic Developer

Last edited by ircdirk : 08-11-15 at 09:53 AM.
  Reply With Quote
08-11-15, 01:25 PM   #12
Rythal
Featured Artist
Featured
Join Date: Aug 2012
Posts: 1,458
and the dev version may have bugs since I have not been able to look over the libguildbank to see if it needs the changes made to it that are in the live versions.
  Reply With Quote
08-11-15, 01:37 PM   #13
ircdirk
A Molten Giant
 
ircdirk's Avatar
AddOn Author - Click to view addons
Join Date: Nov 2014
Posts: 823
Originally Posted by Rythal View Post
and the dev version may have bugs since I have not been able to look over the libguildbank to see if it needs the changes made to it that are in the live versions.
There is pull request for updating libguildbank and ace3
__________________
Carbonite and Carbonite Classic Developer
  Reply With Quote
08-11-15, 10:05 PM   #14
Rythal
Featured Artist
Featured
Join Date: Aug 2012
Posts: 1,458
Originally Posted by ircdirk View Post
There is pull request for updating libguildbank and ace3
Yeah, the libguildbank i'm using was modified to remove the problems and errors when you were not in a guild, so if it's been updated I'll need to either make those same fixes / test if the bugs still exist in the new release.

Hopefully I can afford a new computer soon so I can get back into things, The money I had saved for it just went down the drain when my employer decided I need to move. Moving date is this upcoming Tuesday then I can go back to working on replacing the dead PC.

Last edited by Rythal : 08-11-15 at 10:09 PM.
  Reply With Quote

WoWInterface » Featured Projects » Carbonite » Carbonite: Dev Talk » ERROR..\AddOns\Carbonite.Warehouse\NxWarehouse.lua line 3434:

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off