[Help] Learning to Coding - How to pass var for Global Mods and other items

Pro

Sheet Author

API Scripter

That's a lot to take in, and I'm not familiar with the sheet, and this kind of thing is dependent on knowledge of the sheet. But it sounds like you'll need to know the attribute names for each of the sttas you mention. These sound like they come from a repeating section:  Eldritch Invocation: Agonizing Blast (damage mod),  Eldritch Spear (range mod), [Grasp of Hadar, Lance of Lethargy, Repelling Blast  So you'll need a function to search the repeating section and identify the repeating row that a particular spell is in, by searching the name field in each row the section, and then once you have the row you can get the values you need. Your situation is trickier than it could because it looks like you are grabbing the values of multiple different attributes within the section (range mod, damage mod, etc). I dont know how eldritch invocation works, so i have no idea how to go about doing what you need. If you can describe what you need to do, in extremely specific, point by point detail, we can point you in the right direction. (Or some scripter who knows the sheet and how the eldritch invocation ability works might be able to do it much quicker!).

1595264139

Edited 1595264645

Thanks Gigs for the reply. I not really sure what I need. My thought was to 1. create a variables that would store the names of all the names of their traits.  Not sure how to code that but thought was var traits = something something in repeating traits area ("name").toLowerCase(); var ebTraits = ""; then some kind of loop looking for each var in 'traits'  and you could have more than one Agonizing Blast,( When you cast  Eldritch Blast , add your Charisma modifier to the damage it deals on a hit. ) Eldritch Spear, ( When you cast  Eldritch Blast , its range is 300 feet.) Grasp of Hadar, ( Once on each of your turns when you hit a creature with your  Eldritch Blast , you can move that creature in a straight line 10 feet closer to yourself.) Lance of Lethargy, ( Once on each of your turns when you hit a creature with your  Eldritch Blast , you can reduce that creature’s speed by 10 feet until the end of your next turn.) Repelling Blast ( When you hit a creature with  Eldritch Blast , you can push the creature up to 10 feet away from you in a straight line.) then if these are found store them in ebTraits then pass ebTraits to through a loop adding appropriate text to the corresponding trait  something like  //will need to set some variable not already added on org script var damage = "[[1d10]]"; var chr = Number(getAttrByName(charID, 'charisma_mod')); switch(ebTraits){ case "agonizing blast": damage ="[[1d10+${chr}]]"; } functionality I am uncertain but looking for any code of text that would help Thanks, Tye

1595265083

Pro

Sheet Author

API Scripter

Tye said: 1. create a variables that would store the names of all the names of their traits.  Not sure how to code that but thought was var traits = something something in repeating traits area ("name").toLowerCase(); var ebTraits = ""; You can grab all the attributes for a repeating section using a roll20 function findObjs, and a javascript function filter, something like this var sectionAttributes = findObjs ({ _characterid: charID, _type: "attribute" , }).filter(att => att.get("name").startsWith('repeating_traits_')); The findObjs returns an array of all attributes on charID, then filter keeps only those that start with repeating_traits_. So you end up with all the attributes in the repeating section, on every row. (I'm half-asleep but I think that's the correct code!) You can then loop through those attributes to find the ones you need, or create new variables with data organised however you need it. Whatever you want.

1595276739

GiGs said: You can grab all the attributes for a repeating section using a roll20 function findObjs, and a javascript function filter, something like this var sectionAttributes = findObjs ({ _characterid: charID, _type: "attribute" , }).filter(att => att.get("name").startsWith('repeating_traits_')); The findObjs returns an array of all attributes on charID, then filter keeps only those that start with repeating_traits_. So you end up with all the attributes in the repeating section, on every row. (I'm half-asleep but I think that's the correct code!) You can then loop through those attributes to find the ones you need, or create new variables with data organised however you need it. Whatever you want.  Get some rest Gigs Thanks for the code: not sure its pulling right added this to my code: var sectionAttributes = findObjs({ _characterid: charID, _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); log(sectionAttributes) and I get this in log [{"name":"repeating_traits_-LyKFcooHETThDQncEMq_name","current":"Dark One's Blessing","max":"","_id":"-LyKFcrq3ULZIhZfzI_r","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcooHETThDQncEMq_description","current":"Starting at 1st level, when you reduce a hostile creature to 0 hit points, you gain temporary hit points equal to your Charisma modifier + your warlock level (minimum of 1).","max":"","_id":"-LyKFcrxDD33qmaiKYdE","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcooHETThDQncEMq_source","current":"Class","max":"","_id":"-LyKFcs4kiZiPnHDj3yH","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcooHETThDQncEMq_source_type","current":"The Fiend","max":"","_id":"-LyKFcsCvjZrdtDOkeol","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcooHETThDQncEMq_options-flag","current":"0","max":"","_id":"-LyKFcsK4Suazg6ii5E9","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcooHETThDQncEMq_display_flag","current":"on","max":"","_id":"-LyKFcsT3r7OGO8kcTES","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9T9_name","current":"Lucky","max":"","_id":"-LyKFcsq9YEroKkhT51B","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9T9_description","current":"When you roll a 1 on an attack roll, ability check, or saving throw, you can reroll the die and must use the new roll.","max":"","_id":"-LyKFct-Ks4Ey-PmY8a8","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9T9_source","current":"Racial","max":"","_id":"-LyKFct6lvWjUN4i-pu7","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9T9_source_type","current":"Halfling","max":"","_id":"-LyKFctHV0fJ1ourS_Oa","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9T9_options-flag","current":"0","max":"","_id":"-LyKFctPkt45sz9I65ft","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9T9_display_flag","current":"on","max":"","_id":"-LyKFctXs2Hlr8IIxhmH","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_name","current":"Brave","max":"","_id":"-LyKFctdTXnHmEgYwfWx","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_description","current":"You have advantage on saving throws against being frightened.","max":"","_id":"-LyKFctlIwZsx49I-g3A","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_source","current":"Racial","max":"","_id":"-LyKFcttCDPAzh_mnYdg","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_source_type","current":"Halfling","max":"","_id":"-LyKFcu1o6EZnY1NupN9","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_options-flag","current":"0","max":"","_id":"-LyKFcu9T9ssepkrffu5","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_display_flag","current":"on","max":"","_id":"-LyKFcuHeh3HYevkfmR3","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcoqB6zt6VFMA21j_name","current":"Halfling Nimbleness","max":"","_id":"-LyKFcuQRBLSLQfmBfIx","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcoqB6zt6VFMA21j_description","current":"You can move through the space of any creature that is of a size larger than yours.","max":"","_id":"-LyKFcuZ64eYrISeM1Fs","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcoqB6zt6VFMA21j_source","current":"Racial","max":"","_id":"-LyKFcughXzOeji4LALX","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcoqB6zt6VFMA21j_source_type","current":"Halfling","max":"","_id":"-LyKFcusep07EUGiAkKv","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcoqB6zt6VFMA21j_options-flag","current":0,"max":"","_id":"-LyKFcvNRMZ1r8TG20kv","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcoqB6zt6VFMA21j_display_flag","current":"on","max":"","_id":"-LyKFcvXTc5rM-Sc47BF","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcosR2XC56VO19eE_name","current":"Naturally Stealthy","max":"","_id":"-LyKFcweFS_6MsYzOc7Q","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcosR2XC56VO19eE_description","current":"You can attempt to hide even when you are obscured only by a creature that is at least one size larger than you.","max":"","_id":"-LyKFcwmeYD-74wUaYbR","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcosR2XC56VO19eE_source","current":"Racial","max":"","_id":"-LyKFcww_p95mcFhQES1","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcosR2XC56VO19eE_source_type","current":"Lightfoot Halfling","max":"","_id":"-LyKFcx7rE9I7xPr3HUS","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcosR2XC56VO19eE_options-flag","current":"0","max":"","_id":"-LyKFcxHt-DW3THCwu3V","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcosR2XC56VO19eE_display_flag","current":"on","max":"","_id":"-LyKFcxQNRhCLcY9ePw3","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcouB4TnZllgaq1k_name","current":"Criminal Contact","max":"","_id":"-LyKFcyJsZst_nwzG57L","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcouB4TnZllgaq1k_description","current":"You have a reliable and trustworthy contact who acts as your liaison to a network of other criminals. You know how to get messages to and from your contact, even over great distances; specifically, you know the local messengers, corrupt caravan masters, and seedy sailors who can deliver messages for you.","max":"","_id":"-LyKFcyWYihRm1_ebEZf","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcouB4TnZllgaq1k_source","current":"Background","max":"","_id":"-LyKFcydADInc8m4pn3c","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcouB4TnZllgaq1k_source_type","current":"Criminal","max":"","_id":"-LyKFcylpZsgE7c18zGr","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcouB4TnZllgaq1k_options-flag","current":"0","max":"","_id":"-LyKFcysYMQqOhHsAIbW","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKFcouB4TnZllgaq1k_display_flag","current":"on","max":"","_id":"-LyKFcz-UyG-8ICfwjY9","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDTQ3cagDYQTZC5_name","current":"Eldritch Invocations","max":"","_id":"-LyKHfFakbMNUEU_IMRl","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDTQ3cagDYQTZC5_description","current":"In your study of occult lore, you have unearthed eldritch invocations, fragments of forbidden knowledge that imbue you with an abiding magical ability. At 2nd level, you gain two eldritch invocations of your choice. Your invocation options are detailed at the end of the class description. When you gain certain warlock levels, you gain additional invocations of your choice, as shown in the Invocations Known column of the Warlock table. Additionally, when you gain a level in this class, you can choose one of the invocations you know and replace it with another invocation that you could learn at that level.","max":"","_id":"-LyKHfFyWSo5p8yfSsJT","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDTQ3cagDYQTZC5_source","current":"Class","max":"","_id":"-LyKHfGBReUeBeQbs5kU","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDTQ3cagDYQTZC5_source_type","current":"Warlock","max":"","_id":"-LyKHfGPwGhHmpSy4XAE","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDTQ3cagDYQTZC5_options-flag","current":"0","max":"","_id":"-LyKHfGbVjdCa9O2G10H","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDTQ3cagDYQTZC5_display_flag","current":"on","max":"","_id":"-LyKHfGpBfmiQCcmbWiM","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v3_name","current":"Eldritch Invocation: Agonizing Blast","max":"","_id":"-LyKHfH19e9F-FQ-Hc7_","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v3_description","current":"When you cast eldritch blast, add your Charisma modifier to the damage it deals on a hit.","max":"","_id":"-LyKHfHn_vZ1BxmLrAhp","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v3_source","current":"Class","max":"","_id":"-LyKHfI3hXM9CXJ1Tmcx","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v3_source_type","current":"Warlock","max":"","_id":"-LyKHfId1-wPup9zlVKb","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v3_options-flag","current":"0","max":"","_id":"-LyKHfIzPFq2PoiYd1zX","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v3_display_flag","current":"0","max":"","_id":"-LyKHfJOi1kQDppduPY9","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v4_name","current":"Eldritch Invocation: Repelling Blast","max":"","_id":"-LyKHfJl87TP0Av2DQfg","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v4_description","current":"When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line.","max":"","_id":"-LyKHfK4KEpD16TzMmdi","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v4_source","current":"Class","max":"","_id":"-LyKHfKJRcnBTbk1mX_p","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v4_source_type","current":"Warlock","max":"","_id":"-LyKHfKZNZrBfD5-Sbzi","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v4_options-flag","current":"0","max":"","_id":"-LyKHfKkYQL_ER3Tax19","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKHfDXsYkgMaTbh3v4_display_flag","current":"on","max":"","_id":"-LyKHfKz-wN_ZPfFil3w","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKKn5J7lpJI0U9eO1Y_name","current":"Pact of the Tome","max":"","_id":"-LyKKn7rHfeQkAAGLrGy","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKKn5J7lpJI0U9eO1Y_description","current":"Your patron gives you a grimoire called a Book of Shadows. When you gain this feature, choose three cantrips from any class’s spell list. While the book is on your person, you can cast those cantrips at will. They don’t count against your number of cantrips known. If you lose your Book of Shadows, you can perform a 1-hour ceremony to receive a replacement from your patron. This ceremony can be performed during a short or long rest, and it destroys the previous book. The book turns to ash when you die.","max":"","_id":"-LyKKn8A0_YthPkksej2","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKKn5J7lpJI0U9eO1Y_source","current":"Class","max":"","_id":"-LyKKn8dTZGZmuUlOxiN","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKKn5J7lpJI0U9eO1Y_source_type","current":"Warlock","max":"","_id":"-LyKKn8xFuhT2FdPJDSu","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKKn5J7lpJI0U9eO1Y_options-flag","current":"0","max":"","_id":"-LyKKn9tAhkI_puUGTMf","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKKn5J7lpJI0U9eO1Y_display_flag","current":"on","max":"","_id":"-LyKKnA93-g3jvP2NWEO","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKMzgmRVxudVAOn5uP_name","current":"Eldritch Invocation: Devil’s Sight","max":"","_id":"-LyKMzjLRYgfjkoM0slX","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKMzgmRVxudVAOn5uP_description","current":"You can see normally in darkness, both magical and nonmagical, to a distance of 120 feet.","max":"","_id":"-LyKMzkEtA3VLH_qbNGJ","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKMzgmRVxudVAOn5uP_source","current":"Class","max":"","_id":"-LyKMzkvzJLrhp0Ennxb","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKMzgmRVxudVAOn5uP_source_type","current":"Warlock","max":"","_id":"-LyKMzlKm_dW5UkMWGFN","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKMzgmRVxudVAOn5uP_options-flag","current":"0","max":"","_id":"-LyKMzleFlDKpK9ij4i7","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-LyKMzgmRVxudVAOn5uP_display_flag","current":"on","max":"","_id":"-LyKMzmdTP4UgiBz-0Ne","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-M5j4B0xHpQgIo72sARo_name","current":"Dark One’s Own Luck","max":"","_id":"-M5j4B5RddOAA8yl8Lc9","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-M5j4B0xHpQgIo72sARo_description","current":"Starting at 6th level, you can call on your patron to alter fate in your favor. When you make an ability check or a saving throw, you can use this feature to add a d10 to your roll. You can do so after seeing the initial roll but before any of the roll’s effects occur. Once you use this feature, you can’t use it again until you finish a short or long rest.","max":"","_id":"-M5j4B5eBrXEX-UIn8mE","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-M5j4B0xHpQgIo72sARo_source_type","current":"The Fiend","max":"","_id":"-M5j4B5suehPLXSSEzva","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-M5j4B0xHpQgIo72sARo_source","current":"Class","max":"","_id":"-M5j4B6359Ksc7Dcn1Ab","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-M5j4B0xHpQgIo72sARo_options-flag","current":"0","max":"","_id":"-M5j4B6EFpzl1uYf1ytX","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"},{"name":"repeating_traits_-M5j4B0xHpQgIo72sARo_display_flag","current":"on","max":"","_id":"-M5j4B6R5LxsEPFL8GQ2","_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"}]

1595282749

Edited 1595283862

David M.

Pro

API Scripter

Not to step on any toes, but going on the assumption that GiGs is sleeping and since it's only around 6PM in my time zone, I'll take a stab at your last question that might get you on the right track. It looks like you successfully found all of your repeating trait objects (each one in between curly brackets in your log, and each consisting of some number of property:value pairs) and they are now in your sectionAttributes array (with each object element separated by commas in your log).  Now all you need to do is just loop through the objects in that array like GiGs suggested. Using the _.each method will go through each object in your array (one at a time, in order) and "do something" with the information contained in your objects.  var sectionAttributes = findObjs({                                 _characterid: charID,   _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); _.each(sectionAttributes , function(obj) {       //"do something" with each obj }); BTW, I'm just a noob, too, so I'll yield the floor back to GiGs :) 

1595305317

Pro

Sheet Author

API Scripter

David is correct. That log is showing you an array of attribute objects.  An array is a comma separated list of things, and it can include anything. So an array of every even number 2 and 10 would look like [2, 4, 6, 8, 10]. You could then loop through them using functions like _.each, or do things with it like the filter function i already used. So if you wanted to grab every number that was above 6, you could do let mynewset = [2, 4, 6, 8, 10].filter(num => num >6); that would give you an array like this [8,10] Before I continue, let me say that you are embarking on a pretty complex task. This should give you a taste of the complexity. Don't let it put you off, we are here to help. Now what that sectionAttributes array gives is an array of attribute objects, not numbers, and each attribute object looks like this (picking one out at random): {"name":"repeating_traits_-LyKFcop80oHUmrJF9TA_name","current":"Brave","max":"","_id":"-LyKFctdTXnHmEgYwfWx", "_type":"attribute","_characterid":"-Ly2-LwB7XAjc7-qstEi"}, Everything inside the curly brackets there is one attribute. Notice it has a name a "current" value, a "max" value, an "id", a "type" (attribute), and a "characterid" (which character this attribute belongs to). The underscores at the start of some of property names tells you its a readonly property - you cant change it. Any examples I write assume you have already done this to get the initial array: let sectionAttributes = findObjs({                                 _characterid: charID,   _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); What you need from here is a list of unique row ids, so you can loop through the rows. This weird little function will do that: let rowids = sectionAttributes.map(att => att.get('name').split('_')[2])         .filter((value, index, self) => self.indexOf(value) === index); There's a lot going on here. Looking at the first line: .map()  is a function that loops through an array. In this case, each attribute is - one at a time - assigned to att , then att is passed to this function: att.get('name').split('_')[2] When you want a value out of an attribute object you have to use the .get('name') function. So with each attribute, this gets the name of that attribute, which will be something like: "repeating_traits_-MCjJ2DC6r1iJpFKqzQH_name" that is the attribute name, from the character sheet. The row id is always in the same place, between the 2nd and third underscores, so you can get it by using .split('_')[2] since split gives an array, and [2] picks the 3rd item out of that array. Whew, so at this point we have a list of all the ids. BUT many attributes or on the same row, so there'll be a lot of duplicates, That's where the second line comes in: .filter((value, index, self) => self.indexOf(value) === index); Filter essentially loops through an array, and you can pass one or more properties of each item in the array. value is the item's value, like " -MCjJ2DC6r1iJpFKqzQH " index  is the items position in the array, which will be a number like 0, 1, 2, etc. self  is the actual array your working with. In this function you need to be able to act on that. So for each id in the array, we do this self.indexOf(value) === index) When doing indexOf, it returns the index of the first  matching item. Now in this array, which may have a row id at indexes of 0, 1, 2, 3, and 4, because every attribute on the same row has the same id. So, this checks: is the index of the item we are checking now equal to the very first matching item. If it is, thats true,  so we keep it. If its not (it's the second or later instance of that same row id), we discard it. So with this function we now have a unique list of row ids. So putting those together: let sectionAttributes = findObjs({                                 _characterid: charID,   _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); let rowids = sectionAttributes.map(att => att.get('name').split('_')[2])         .filter((value, index, self) => self.indexOf(value) === index); You can now get the value of any attribute in the repeating section, and can loop through the rows. Using these two arrays, and some of the techniques used to build them (like .get(), filter and the split('_')[2] trick), you can do whatever you need to with a repeating section. for some things, you'll want to loop through the entire set of attributes, for others you'll want to loop throgh the unique ids and check specific attributes on each row, by building up attribute names, like let attr_name = 'repeating_trait_' + rowid + '_whatever-attribute-i-want-to-check'; that would be inside a loop, like rowids.forEach(rowid => {     let attr_name = 'repeating_trait_' + rowid + '_whatever-attribute-i-want-to-check';     // do something        return result-of-operation }); Anyway, this very complex, so good luck and if you need more help ask away.

1595313093

Edited 1595315257

Thank you both so much!  I have sent some time working on this,  I will post my updated code below  This code was what I tried on my own before I saw your posts I will look and study what you have suggested As of Now I have managed to get the 5 traits I am looking to post the correct info in the template.  The only problem is that I have 3 items that will effect the desc field.  Since you may have one or all 3 I need to append the desc field but that is not working var desc = "" I thought that desc+= would append text but I feel that is not the case here var sectionAttributes = findObjs({ _characterid: charID, _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); _.each(sectionAttributes, function(attCurrentName){ attCurrentName = attCurrentName.get("current"); //log(attCurrentName) switch(attCurrentName){ case "Eldritch Invocation: Agonizing Blast": damage = `[[1d10+${chr}[Chr]]]`; sendChat("EB","/w gm Agonizing Blast found"); break; case "Eldritch Invocation: Eldritch Spear": range = "300" sendChat("EB","/w gm Eldritch Spear found"); break; case "Eldritch Invocation: Grasp of Hadar": desc = `{{desc=${graspOfHadar}`; sendChat("EB","/w gm Grasp of Hadar found"); break; case "Eldritch Invocation: Lance of Lethargy": if(desc === ""){ desc = `{{desc=${lanceOfLethargy}`; } else { desc+=` +${lanceOfLethargy}`; }; sendChat("EB","/w gm Lance of Lethargy found"); break; case "Eldritch Invocation: Repelling Blast": if(desc === ""){ desc = `{{desc=${repellingBlast}`; } else { desc+=` +${repellingBlast}`; }; sendChat("EB","/w gm Repelling Blast found"); break; default: //log(attCurrentName+" not EB Item"); break; }; }); if(desc !== ""){ desc +="}}"}; EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc); break; here is the EB function  function EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc){ var numberOfBeams = (Math.floor((casterLevel + 1)/6)+1); var beamOutput = ""; //Having issues here where chat populates the beam # out of order for(counter=1;counter<=numberOfBeams;counter++){ var outputMessage = `&{template:atkdmg} {{mod=${spellAtkBonus}}} {{rname=Eldritch Blast ${counter}}} {{r1=[[1d20+${spellAtkBonus} [Spell]]]}} ${rtype}+${spellAtkBonus} [Spell]]]}} {{attack=1}} ${desc} {{range=${range}ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=${damage}}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=${characterName}}}`; log(outputMessage) //this if statment sends chat message as the who sent message if (character) sendChat('character|'+character.id, character.get('name')+outputMessage); else sendChat('player|'+player.id, player.get('displayname')+outputMessage); }; here is the log outputMessage "&{template:atkdmg} {{mod=12}} {{rname=Eldritch Blast 1}} {{r1=[[1d20+12 [Spell]]]}} {{normal=1}} {{r2=[[0d20+12 [Spell]]]}} {{attack=1}} {{desc=Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself.}} {{range=120ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=[[1d10+6[Chr]]]}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=Alton Ashfall}}" Here are the 3 "found" send chat items 2:46AM (From EB):  Agonizing Blast found 2:46AM (From EB):  Repelling Blast found 2:46AM (From EB):  Grasp of Hadar found Thus Repelling Blast text is not being added to in the desc text UPDATED CODE as of 7/21/2020 at 0219 EST  on("ready",function(){ log('Custom Spellbook Online'); on("chat:message", function(msg) { if (msg.type == "api" && msg.content.indexOf("!customSpellbook") == 0){ var args = msg.content.split(/\s+/); var spellName = args[1].toLowerCase(); var spellLevel = Number(args[2]); var selected = msg.selected; if (selected===undefined) { sendChat("Custom Spellbook", "/w gm no token selected"); return; } var character = findObjs({ type: 'character', name: msg.who })[0], player = getObj('player', msg.playerid); var tokenid = selected[0]._id; var token = getObj("graphic",tokenid); var charID = token.get("represents"); var characterName = token.get("name"); var casterLevel = Number(getAttrByName(charID, 'level')); var spellAtkBonus = Number(getAttrByName(charID, 'spell_attack_bonus')); var chr = Number(getAttrByName(charID, 'charisma_mod')); var desc = "" var graspOfHadar = "Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself." var lanceOfLethargy = "Once on each of your turns when you hit a creature with your Eldritch Blast, you can reduce that creature’s speed by 10 feet until the end of your next turn." var repellingBlast = "When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line." switch(spellName){ case "magicmissile": magicMissile(spellLevel); break; case "eb": var damage = "[[1d10]]"; var range = "120" var rtype = args[3]; if (rtype === undefined) { sendChat("CSB - Eldritch Blast", "No Roll type provided, Please state **Normal, Advantage, or Disadvantage**"); return; } rtype = args[3].toLowerCase(); switch(rtype){ case "normal": rtype ="{{normal=1}} {{r2=[[0d20"; break; case "advantage": rtype = "{{advantage=1}} {{r2=[[1d20ro"; break; case "disadvantage": rtype = "{{disadvantage=1}} {{r2=[[1d20ro"; break; default: sendChat("CSB - Eldritch Blast", "Roll type not properly identified, Please state either **Normal, Advantage, or Disadvantage**"); return; } var sectionAttributes = findObjs({ _characterid: charID, _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); _.each(sectionAttributes, function(attCurrentName){ attCurrentName = attCurrentName.get("current"); //log(attCurrentName) switch(attCurrentName){ case "Eldritch Invocation: Agonizing Blast": damage = `[[1d10+${chr}[Chr]]]`; sendChat("EB","/w gm Agonizing Blast found"); break; case "Eldritch Invocation: Eldritch Spear": range = "300" sendChat("EB","/w gm Eldritch Spear found"); break; case "Eldritch Invocation: Grasp of Hadar": desc = `{{desc=${graspOfHadar}`; sendChat("EB","/w gm Grasp of Hadar found"); break; case "Eldritch Invocation: Lance of Lethargy": if(desc === ""){ desc = `{{desc=${lanceOfLethargy}`; } else { desc+=` +${lanceOfLethargy}`; }; sendChat("EB","/w gm Lance of Lethargy found"); break; case "Eldritch Invocation: Repelling Blast": if(desc === ""){ desc = `{{desc=${repellingBlast}`; } else { desc+=` +${repellingBlast}`; }; sendChat("EB","/w gm Repelling Blast found"); break; default: //log(attCurrentName+" not EB Item"); break; }; }); if(desc !== ""){ desc +="}}"}; EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc); break; } //Nick's code to reduce used spells (turned off until I know where to put this since I've added cantrips i.e. Eldritch Blast) //sendChat("API",`!modbattr --charid @{${charID}} --lvl${spellLevel}_slots_expended|-1`); //sendChat("API",`!modbattr --charid @{${charID}} --SpellsCasted|+1`); } }); function magicMissile(spellLevel){ var numberOfDarts = spellLevel + 2; var dartOutput = ""; for(counter=1;counter<=numberOfDarts;counter++){ dartOutput+=`{{Dart ${counter} = [[1d4+1]]}}`; } var outputMessage = `&{template:default} {{name=Magic Missile}} ${dartOutput}`; sendChat("API",outputMessage); }; function EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc){ var numberOfBeams = (Math.floor((casterLevel + 1)/6)+1); var beamOutput = ""; //Having issues here where chat populates the beam # out of order for(counter=1;counter<=numberOfBeams;counter++){ var outputMessage = `&{template:atkdmg} {{mod=${spellAtkBonus}}} {{rname=Eldritch Blast ${counter}}} {{r1=[[1d20+${spellAtkBonus} [Spell]]]}} ${rtype}+${spellAtkBonus} [Spell]]]}} {{attack=1}} ${desc} {{range=${range}ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=${damage}}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=${characterName}}}`; log(outputMessage) //this if statment sends chat message as the who sent message if (character) sendChat('character|'+character.id, character.get('name')+outputMessage); else sendChat('player|'+player.id, player.get('displayname')+outputMessage); //sendChat(characterName,outputMessage); }; }; });

1595315485

Pro

Sheet Author

API Scripter

What happens with desc +=? Does the script crash or does it just fail to return the expected value? desc += whatever; should work. But if it doesnt, try this instead: desc = desc + whatever; or even desc = `${desc} ${whatever}`; In any case, I would recommend removing the {{desc= part from your loop. Running through the loop adding the bits together, then after the loop, do something like desc = desc ? `{{desc=${desc} }}` : ''; that avoids the need to test if desc == '' several times during the loop. Note that the above is a ternary operator, its an if statement in one line. It's the same as doing: if(desc) {     desc = `{{desc=${desc} }}`; } else {     desc = ''; } Just a lot more compact. if(desc) will be true if desc has been assigned a value, and false if it is still ''.

1595316481

Pro

Sheet Author

API Scripter

I see your note about EB sending chat messages out of order. The way to avoid that is to build all the text into one string, and send just one sendChat message. In fact you should build all output from the entire script into one message, and just use one sendChat to send it.  Here's the function rewritten to send one chat message: function EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc){     var numberOfBeams = (Math.floor((casterLevel + 1)/6)+1);     var beamarray = [];     var outputMessage = `&{template:atkdmg} {{mod=${spellAtkBonus}}} {{rname=Eldritch Blast ${counter}}} {{r1=[[1d20+${spellAtkBonus} [Spell]]]}} ${rtype}+${spellAtkBonus} [Spell]]]}} {{attack=1}} ${desc} {{range=${range}ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=${damage}}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=${characterName}}}`;     //this makes an array with as mean copies of the outputmeassage as needed     for(counter=1;counter<=numberOfBeams;counter++){         beamarray.push(outputMessage);     };     // this converts the array into a string, with newlines between each on, so they each go on a new line.     var beamOutput = beamarray.join(`\n`);     //this if statment sends chat message as the who sent message     //sendChat(characterName,outputMessage);     if (character) sendChat('character|'+character.id, character.get('name') + beamOutput);     else sendChat('player|'+player.id, player.get('displayname') + beamOutput); };     Though it really should end with:     var beamOutput = beamarray.join(`\n`);     //return this to the calling script, toi then be included with other chat messages into one.     return beamOutput; };     and in your switch case you'd call it with something like var message = EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc); and can then combine it with other message strings, to create a single output message.

1595316966

GiGs said: What happens with desc +=? Does the script crash or does it just fail to return the expected value? desc += whatever; should work. But if it doesnt, try this instead: desc = desc + whatever; or even desc = `${desc} ${whatever}`; It fails to return a value with any of the 3: You lost me with this: In any case, I would recommend removing the {{desc= part from your loop. Running through the loop adding the bits together, then after the loop, do something like desc = desc ? `{{desc=${desc} }}` : ''; that avoids the need to test if desc == '' several times during the loop. Note that the above is a ternary operator, its an if statement in one line. It's the same as doing: if(desc) {     desc = `{{desc=${desc} }}`; } else {     desc = ''; } Just a lot more compact. if(desc) will be true if desc has been assigned a value, and false if it is still ''.

1595317611

GiGs said: I see your note about EB sending chat messages out of order. The way to avoid that is to build all the text into one string, and send just one sendChat message. In fact you should build all output from the entire script into one message, and just use one sendChat to send it.  Here's the function rewritten to send one chat message: function EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc){     var numberOfBeams = (Math.floor((casterLevel + 1)/6)+1);     var beamarray = [];     var outputMessage = `&{template:atkdmg} {{mod=${spellAtkBonus}}} {{rname=Eldritch Blast ${counter}}} {{r1=[[1d20+${spellAtkBonus} [Spell]]]}} ${rtype}+${spellAtkBonus} [Spell]]]}} {{attack=1}} ${desc} {{range=${range}ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=${damage}}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=${characterName}}}`;     //this makes an array with as mean copies of the outputmeassage as needed     for(counter=1;counter<=numberOfBeams;counter++){         beamarray.push(outputMessage);     };     // this converts the array into a string, with newlines between each on, so they each go on a new line.     var beamOutput = beamarray.join(`\n`);     //this if statment sends chat message as the who sent message     //sendChat(characterName,outputMessage);     if (character) sendChat('character|'+character.id, character.get('name') + beamOutput);     else sendChat('player|'+player.id, player.get('displayname') + beamOutput); };     Though it really should end with:     var beamOutput = beamarray.join(`\n`);     //return this to the calling script, toi then be included with other chat messages into one.     return beamOutput; };     and in your switch case you'd call it with something like var message = EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc); and can then combine it with other message strings, to create a single output message. I updated Switch Case to your var message..... Updated EB function when no errors on save, get a    ReferenceError: counter is not defined when running the API command

1595318674

Pro

Sheet Author

API Scripter

weird that it doesn't work now but apparently worked before? It's lacking a variable declaration for counter, so change the for line to  for(let counter=1;counter<=numberOfBeams;counter++){

1595318998

Edited 1595319951

GiGs said: weird that it doesn't work now but apparently worked before? It's lacking a variable declaration for counter, so change the for line to  for(let counter=1;counter<=numberOfBeams;counter++){ Nope same error

1595319150

Pro

Sheet Author

API Scripter

Tye said: You lost me with this: In any case, I would recommend removing the {{desc= part from your loop. Running through the loop adding the bits together, then after the loop, do something like desc = desc ? `{{desc=${desc} }}` : ''; that avoids the need to test if desc == '' several times during the loop. Note that the above is a ternary operator, its an if statement in one line. It's the same as doing: if(desc) {     desc = `{{desc=${desc} }}`; } else {     desc = ''; } Just a lot more compact. if(desc) will be true if desc has been assigned a value, and false if it is still ''. I was suggesting something like changing these bits                                    case "Eldritch Invocation: Lance of Lethargy": if(desc === ""){ desc = `{{desc=${lanceOfLethargy}`; } else { desc+=` +${lanceOfLethargy}`; }; sendChat("EB","/w gm Lance of Lethargy found"); break; to                                     case "Eldritch Invocation: Lance of Lethargy": desc+=` +${lanceOfLethargy}`; sendChat("EB","/w gm Lance of Lethargy found"); break; then at the end where you have  if(desc !== ""){ desc +="}}"}; you can change that to  if(desc !== ""){ desc = `{{desc=${desc}}}`; } Though it would be better to use the array method. So just before the each loop starts, create let descarray = []; then the above sections would be like case "Eldritch Invocation: Lance of Lethargy": descarray.push(lanceOfLethargy); sendChat("EB","/w gm Lance of Lethargy found"); break; Then the end part can use .join(' ') to join them and put a line break in each one/ if(desc !== ""){ desc = `{{desc=${descarray.join(' ')} }}`; }

1595319432

Thanks GiGs I will look at it later time to get some zzzz

1595319594

Edited 1595319605

Pro

Sheet Author

API Scripter

Tye said: GiGs said: weird that it doesn't work now but apparently worked before? It's lacking a variable declaration for counter, so change the for line to  for(let counter=1;counter<=numberOfBeams;counter++){ Nope same error Check that the outputMessage on the line before ends properly. The problem might not be with counter, but with improper syntax just before the loop. When you have hard to define errors, one useful thing to do is disable all other scripts, so you can use the line its reporting the error on, though that might not help here. I'm confused why its not working now and was working before, and think some other change in the code might be responsible.

1595319932

I see that counter is in the var outputMessage var outputMessage = `&{template:atkdmg} {{mod=${spellAtkBonus}}} {{rname=Eldritch Blast ${counter}}} which is before the new for(let

1595320198

Pro

Sheet Author

API Scripter

aha, well spotted, i didnt notice you had a count in there. That's the source of the error. Change the function to this: function EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc){     var numberOfBeams = (Math.floor((casterLevel + 1)/6)+1);     var beamarray = [];     var outputMessage = counter => `&{template:atkdmg} {{mod=${spellAtkBonus}}} {{rname=Eldritch Blast ${counter}}} {{r1=[[1d20+${spellAtkBonus} [Spell]]]}} ${rtype}+${spellAtkBonus} [Spell]]]}} {{attack=1}} ${desc} {{range=${range}ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=${damage}}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=${characterName}}}`;     //this makes an array with as many copies of the outputmeassage as needed     for(let counter=1;counter<=numberOfBeams;counter++){         beamarray.push(outputMessage(counter));     };     // this converts the array into a string, with newlines between each on, so they each go on a new line.     var beamOutput = beamarray.join(`\n`);     //this if statment sends chat message as the who sent message     //sendChat(characterName,outputMessage);     if (character) sendChat('character|'+character.id, character.get('name') + beamOutput);     else sendChat('player|'+player.id, player.get('displayname') + beamOutput); };     this changes the outputmessage to a function, that accepts the counter.

1595320388

Pro

Sheet Author

API Scripter

There might still be a problem: What's the purpose of the character.get('name') at the start of the sendchat message: if (character) sendChat('character|'+character.id, character.get('name') + beamOutput); There's no /w  there, so I dont understand why you are putting the character name at the start of the message. 

1595320709

GiGs  said: Though it would be better to use the array method. So just before the each loop starts, create let descarray = []; then the above sections would be like case "Eldritch Invocation: Lance of Lethargy": descarray.push(lanceOfLethargy); sendChat("EB","/w gm Lance of Lethargy found"); break; Then the end part can use .join(' ') to join them and put a line break in each one/ if(desc !== ""){ desc = `{{desc=${descarray.join(' ')} }}`; } the descarry is not adding to the desc variable var desc = "" being sent to EB function notice the extra space in log between {{attack=1}}  {{range=120ft}} that is were the {{desc=}} would go per our outputMessage variable "&{template:atkdmg} {{mod=12}} {{rname=Eldritch Blast 1}} {{r1=[[1d20+12 [Spell]]]}} {{normal=1}} {{r2=[[0d20+12 [Spell]]]}} {{attack=1}} {{range=120ft}} {{damage=1}} {{dmg1flag=1}} {{dmg1=[[1d10+6[Chr]]]}} {{dmg1type=force}} {{crit1=[[10]]}} {{crit2=[[10]]}}{{spelllevel= Cantrip}} {{charname=Alton Ashfall}}"

1595321395

Pro

Sheet Author

API Scripter

We know there's something up with your text values. I dont wee why they aren't working, but I noticed these lines are missing semi-colons at the end:  var desc = ''             var graspOfHadar = 'Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself.'             var lanceOfLethargy = 'Once on each of your turns when you hit a creature with your Eldritch Blast, you can reduce that creature’s speed by 10 feet until the end of your next turn.'             var repellingBlast = 'When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line.' Add a semi colon to the end of each line and corss your fingers that it makes a difference...

1595321818

The fix to the counter error worked added ; to the 3 lines the desc is still not being passed value remains ""

1595321878

Pro

Sheet Author

API Scripter

You should also log those variables to make sure they have the correct values, like case 'Eldritch Invocation: Lance of Lethargy':                                 descarray.push(lanceOfLethargy);                                 sendChat('EB','/w gm Lance of Lethargy found');                                 sendChat('lanceOfLethargy',lanceOfLethargy);                                 log(`Lance is: ${lanceOfLethargy}`);                                 log(descarray);                                 break; This is more log messages than I'd normally use, but you can delete them after checking things have the values they are supposed to.

1595322821

GiGs said: You should also log those variables to make sure they have the correct values, like case 'Eldritch Invocation: Lance of Lethargy':                                 descarray.push(lanceOfLethargy);                                 sendChat('EB','/w gm Lance of Lethargy found');                                 sendChat('lanceOfLethargy',lanceOfLethargy);                                 log(`Lance is: ${lanceOfLethargy}`);                                 log(descarray);                                 break; This is more log messages than I'd normally use, but you can delete them after checking things have the values they are supposed to. (From EB):  Agonizing Blast found 5:12AM (From EB):  Repelling Blast found 5:12AM repellingBlast: When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line. 5:12AM graspOfHada: Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself. 5:12AM (From EB):  Grasp of Hadar found 5:12AM (From EB):  Lance of Lethargy found 5:12AM lanceOfLethargy: Once on each of your turns when you hit a creature with your Eldritch Blast, you can reduce that creature’s speed by 10 feet until the end of your next turn. "Repelling Blast is: When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line." ["When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line."] "Grasp of Hadar is: Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself." ["When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line.","Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself."] "Lance is: Once on each of your turns when you hit a creature with your Eldritch Blast, you can reduce that creature’s speed by 10 feet until the end of your next turn." ["When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line.","Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself.","Once on each of your turns when you hit a creature with your Eldritch Blast, you can reduce that creature’s speed by 10 feet until the end of your next turn."]

1595322892

Those work but not passing into desc

1595326243

Pro

Sheet Author

API Scripter

at least we know the variables are being passed properly, and the descarray is being built properly. I just realised I didnt change this part to account for using the array: if(desc !== ""){ desc = `{{desc=${descarray.join(' ')} }}`; } It should be if(descarray.length > 0){ desc = `{{desc=${descarray.join(' ')} }}`; }

1595343712

GiGs said: at least we know the variables are being passed properly, and the descarray is being built properly. I just realised I didnt change this part to account for using the array: if(desc !== ""){ desc = `{{desc=${descarray.join(' ')} }}`; } It should be if(descarray.length > 0){ desc = `{{desc=${descarray.join(' ')} }}`; } That worked EB is done!!!! YES!!!! Thanks!!!! now on to another variation damage mod LOL will updated org. Post of progress Thank you so much GiGs to add to my Toolbox!!!

1595344220

Pro

Sheet Author

API Scripter

Yay, thats great to hear.

1595385120

Ok Changes I have made today: I have added the desc to be updated for all 5 invocations var sectionAttributes = findObjs({ _characterid: charID, _type: "attribute", }).filter(att => att.get("name").startsWith('repeating_traits_')); let descarray = []; _.each(sectionAttributes, function(attCurrentName){ attCurrentName = attCurrentName.get("current"); switch(attCurrentName){ case "Eldritch Invocation: Agonizing Blast": damage = `[[1d10+${chr}[Chr]]]`; descarray.push(agonizingBlast); break; case "Eldritch Invocation: Eldritch Spear": range = "300" descarray.push(eldritchSpear); break; case "Eldritch Invocation: Grasp of Hadar": descarray.push(graspOfHadar); break; case 'Eldritch Invocation: Lance of Lethargy': descarray.push(lanceOfLethargy); break; case "Eldritch Invocation: Repelling Blast": descarray.push(repellingBlast); break; default: added Global Damage Mod and Type to pass through a for/else to see if damage mod is 0 or has damage dice amounts made dmg2 turn on and off with damage mods.  if no checked damaged mods dmg2 is off.   var globalDamageMod = getAttrByName(charID, 'global_damage_mod_roll'); var dmg2 = []; var globalDamageType = getAttrByName(charID, 'global_damage_mod_type'); here is the for/else added just before calling the EB function if(globalDamageMod == "0") { dmg2 = "{{dmg2flag=}} {{dmg2=}}"; log("True: "+dmg2); } else { dmg2 = `{{dmg2flag=1}} {{dmg2type=${globalDamageType}}} {{dmg2=[[${globalDamageMod}]]}}`; log("false: "+dmg2); } var message = EB(casterLevel, spellAtkBonus, characterName, rtype, damage, range, character, player, desc, dmg2); break; lastly added dmg2 to the outputMessage in the EB function. Updated gitthub with code found here  <a href="https://gist.github.com/TooTallTye/52b32e0095a9562741ffd9fc6be0f7e1" rel="nofollow">https://gist.github.com/TooTallTye/52b32e0095a9562741ffd9fc6be0f7e1</a> updated org. post with todo's need to find code or way add Global Attack Mods

1595477988

Ok I think I have managed to finish all of my tasks except make a marco from an API..... For Eldritch Blast: The running !customSpellbook eb 0 ?{Eldritch Blast Roll Type?|Normal|Advantage|Disadvantage} will cast the correct number of Blasts (will number each blast), add any Attack Modifiers to the attack roll, will also add a 2nd set of damage rolls for any Damage Modifiers. Also roll Normal, Advantage, or at Disadvantage. It will also send the message from who sent the message (chat as), look within the traits for of the 5 Invocations that effect Eldritch Blast and update the spell attack cards as needed.  For each Invocation the  appropriate description will be added into the description field each on it's own line. 1. Eldritch Invocation: Agonizing Blast  When you cast Eldritch Blast, add your Charisma modifier to the damage it deals on a hit. 2. Eldritch Invocation: Eldritch Spear When you cast Eldritch Blast, its range is 300 feet. 3. Eldritch Invocation: Grasp of Hadar Once on each of your turns when you hit a creature with your Eldritch Blast, you can move that creature in a straight line 10 feet closer to yourself. 4. Eldritch Invocation: Lance of Lethargy Once on each of your turns when you hit a creature with your Eldritch Blast, you can reduce that creature’s speed by 10 feet until the end of your next turn. 5. Eldritch Invocation: Repelling Blast When you hit a creature with Eldritch Blast, you can push the creature up to 10 feet away from you in a straight line. Here are 2 images of the chat messages from the API,  The Character Alton Ashfall a Lightfoot Halfling, Pact of the Fiend Warlock 6 with a bless 1d4 attack mod, and Hex 1d6 Narcotic and for ease of showing all 5 invocations. Image 1 Basic Eldritch Blast Lv6 player no Invocations: Image 2 Fully Loaded (Range now 300, Chr added to damage, bless added (the +1 and +4) to attack, hex added in damage 2 block)  

1595480978

Pro

Sheet Author

API Scripter

Nice work!

1595481482

Thanks for your help and advice

1596517875

Hello Everyone, the spells that I have coded in are Scorching Ray, Eldritch Blast, and Magic Missile.  All results appear in chat in the correct damage or atkdmg roll templates for 5e OGL. I have it even looking to see if they are able to cast that level of spells and if they have any available. Which is only needed for Magic Missile and Scorching Ray at the moment. Adds Attack and Damage mods as needed even subtracts the spell when used and displays a message in chat of how many remaining. Here is the link to my code. <a href="https://gist.github.com/TooTallTye/52b32e0095a9562741ffd9fc6be0f7e1" rel="nofollow">https://gist.github.com/TooTallTye/52b32e0095a9562741ffd9fc6be0f7e1</a> What I want to know is how can I streamline the code,  not sure how to convey it but to remove redundant bits of code as well as clean up the formatting, make if more readable to the coders who would read this.  Everything works.  I want to add more spells but keep it simple. Please advise. Thanks Tye

1596518153