ToME: the Tales of Maj'Eyal

Everything about ToME
It is currently Mon Mar 01, 2021 5:10 pm

All times are UTC




Post new topic Reply to topic  [ 444 posts ]  Go to page Previous  1 ... 26, 27, 28, 29, 30
Author Message
PostPosted: Wed Dec 23, 2020 10:45 pm 
Offline
Sher'Tul Godslayer

Joined: Thu Jan 23, 2003 8:13 pm
Posts: 1993
Location: A shallow water area south of Bree
Rukaneo wrote:
But now I encountered a new one when comes to Filleral in Cave upon first arriving through the portal. This happpens when you speak to her and starting the new quest in the East for Staff of absorption.

[sound F/X: source diving] Hmm, that actually looks like a bug in Ignore Race/Class Locks; it appears to be tripping over a missing race descriptor. What race and class is the character that's having this error?

_________________
"Blessed are the yeeks, for they shall inherit Arda..."


Top
 Profile  
 
PostPosted: Thu Dec 24, 2020 6:40 pm 
Offline
Low Yeek

Joined: Sat Dec 19, 2020 7:14 pm
Posts: 8
Quote:
[sound F/X: source diving] Hmm, that actually looks like a bug in Ignore Race/Class Locks; it appears to be tripping over a missing race descriptor. What race and class is the character that's having this error?


Ahh the race I am currrently. Well I am using is an addon race with an addon race prodigy. Astralinferno's Zilquish Race with Recaiden Dreadmaster prodidgy with turns you into a dread race.


Top
 Profile  
 
PostPosted: Thu Dec 24, 2020 7:02 pm 
Offline
Uruivellas

Joined: Mon Jan 27, 2014 10:40 am
Posts: 754
oh that explains everything, now that I think about it!

So Ignore R/C locks is checking the player's subrace and then checking the birther definition for that subrace to see if they match; in order to trigger the special achievement you added.
(It might also be checking the birther definition in other places; this was the one I found)

The Dreadmaster race evolution changes your subrace to Dread.
Unlike the Lich evolution, there is no birther definition for Dread.

ergo, its Nil, and when you try and look at the nonexistent Locked text, it errors.
(Presumably.)

I'm suggesting to rec that they should add a birth definition to Dread, since this is more their error than it is yours, but accounting for nil results here would future proof.

Hope I didnt come in and tell you this after you'd already spent time looking into it.


Top
 Profile  
 
PostPosted: Fri Dec 25, 2020 3:19 am 
Offline
Sher'Tul Godslayer

Joined: Thu Jan 23, 2003 8:13 pm
Posts: 1993
Location: A shallow water area south of Bree
astralInferno wrote:
So Ignore R/C locks is checking the player's subrace and then checking the birther definition for that subrace to see if they match; in order to trigger the special achievement you added.
(It might also be checking the birther definition in other places; this was the one I found)

Yes, that's definitely where the problem is, but more proximately, the specific code that's choking isn't actually my code; it's buried in the Birther dialog, and it's completely unnecessary for our purposes. I'd have removed it earlier if I'd known it would cause problems. I've pushed out a new release that should fix the problem; We Apologize for the Inconvenience.™

(EDIT: Waaiit a minute… [sound F/X: source diving] Dangit, you're right, I am checking the descriptors myself too; I'm going to have to patch over that. Gimme a minute… EDIT 2: Okay, that should fix it. [Famous last words… :oops: ])

In the meantime, pushing out v17b.3 for both, which is mostly a combatibility release for use with my new Interact with Sidekicks addon by way of Talent Point Planner, but also includes updates for Enhanced Object Compare and Go to Landmark.

_________________
"Blessed are the yeeks, for they shall inherit Arda..."


Top
 Profile  
 
PostPosted: Sat Dec 26, 2020 3:59 pm 
Offline
Low Yeek

Joined: Sat Dec 19, 2020 7:14 pm
Posts: 8
Thank you Zizzo! You're an awesome addon maker. 8) 8) :D :D


Top
 Profile  
 
PostPosted: Tue Dec 29, 2020 12:56 am 
Offline
Sher'Tul Godslayer

Joined: Thu Jan 23, 2003 8:13 pm
Posts: 1993
Location: A shallow water area south of Bree
Rukaneo wrote:
Thank you Zizzo! You're an awesome addon maker. 8) 8) :D :D

*blush* (bows)

In the meantime, pushing out v17b.4 for both, with bugfixes in Talent Point Planner and a small new feature in Go to Landmark.

_________________
"Blessed are the yeeks, for they shall inherit Arda..."


Top
 Profile  
 
PostPosted: Fri Jan 15, 2021 3:11 pm 
Offline
Thalore

Joined: Sat Sep 27, 2014 11:33 am
Posts: 120
Location: Irkkk
Hi
We have translated ZOmnibus Addon Pack/ZOmnibus Lite in Chinese, as well as fixed a small problem in the translation file.
https://mega.nz/folder/n5R33SwB#QVqlFw5e6Djku-jpGT1uHg
This zip includes some modification to your original addon, so I suggest a diff before using it.
It may need some further fixes in the future, and we are to cooperate with you in the future.


Top
 Profile  
 
PostPosted: Sun Jan 17, 2021 8:04 pm 
Offline
Sher'Tul Godslayer

Joined: Thu Jan 23, 2003 8:13 pm
Posts: 1993
Location: A shallow water area south of Bree
starsapphire wrote:
Hi
We have translated ZOmnibus Addon Pack/ZOmnibus Lite in Chinese, as well as fixed a small problem in the translation file.
https://mega.nz/folder/n5R33SwB#QVqlFw5e6Djku-jpGT1uHg
This zip includes some modification to your original addon, so I suggest a diff before using it.
It may need some further fixes in the future, and we are to cooperate with you in the future.

[sound F/X: source diving] Ouch, I missed a lot of localization spots, apparently. We Apologize for the Inconvenience.™ :oops: Pushing out v17b.5 to cover those. [EDIT: Make that 17b.6. I thought that unilateral change from e.name to e:getName() in Go to Landmark looked suspicious…] (I'll skip making separate announcements for the affected component addons, since there are seventeen of them.)

As for the translations themselves, I think it would work better if you released those as a separate addon. I did a quick mock-up with the translations you provided, and an addon with just the data/locales files appears to work together with ZOmnibus to provide all the necessary translations — and it has the advantage of also working with any or all of the component addons separately. Plus it eliminates an extra step in the update cycle if a translation needs to be changed or added.

_________________
"Blessed are the yeeks, for they shall inherit Arda..."


Top
 Profile  
 
PostPosted: Sun Feb 21, 2021 1:46 am 
Offline
Sher'Tul Godslayer

Joined: Thu Jan 23, 2003 8:13 pm
Posts: 1993
Location: A shallow water area south of Bree
And pushing out v17c for both, with key-related improvements in Go to Landmark and Inventory Keys.

_________________
"Blessed are the yeeks, for they shall inherit Arda..."


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 444 posts ]  Go to page Previous  1 ... 26, 27, 28, 29, 30

All times are UTC


Who is online

Users browsing this forum: No registered users and 2 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
cron
Powered by phpBB® Forum Software © phpBB Group