Skip to content

fix: correct data errors in Tatooine static quest NPC tab files#413

Open
nicolassanchez02 wants to merge 3 commits into
SWG-Source:masterfrom
nicolassanchez02:fix/tatooine-static-quest-npc-data
Open

fix: correct data errors in Tatooine static quest NPC tab files#413
nicolassanchez02 wants to merge 3 commits into
SWG-Source:masterfrom
nicolassanchez02:fix/tatooine-static-quest-npc-data

Conversation

@nicolassanchez02
Copy link
Copy Markdown
Contributor

@nicolassanchez02 nicolassanchez02 commented May 18, 2026

Four data bugs in the npc/static_quest tab files for Tatooine NPCs. Pure data corrections, no logic changes.

rakir_banai.tab: retrieve_object path reads objection/tangible/mission/mission_datadisk.iff on both fetch quest rows. The leading objection is a typo for object. Both fetch quests (Silm'an Voueen and Wizzel) permanently fail because the path is invalid.

kitster_banai.tab: overall_objvar is spelled static.kister_banai (missing t) across all five rows. The smuggle quest row (Thermal Det/slicer) also has blank player_script and npc_script columns so npc.static_quest never attaches and the quest can't start.

tekil_barje.tab: 26 trailing lines after the three valid data rows, blank lines followed by rows labeled 16 - Extras through 20 - Extras. Leftover template scaffolding. Could shift row-count parsing.

blerx_tango.tab: First rescue quest row (Mixon Guncher/bith_sniper) has deliver_object set to object/tangible/mission/mission_datadisk.iff. Rescue quests don't deliver items, this is copy-paste residue.

Closes #411

Four bugs found across the npc/static_quest data tables:

- rakir_banai.tab: fix typo 'objection/tangible/' -> 'object/tangible/'
  in retrieve_object for the two fetch quest rows; the mission datadisk
  path was invalid causing both quests to permanently fail item retrieval.

- kitster_banai.tab: fix objvar typo 'static.kister_banai' ->
  'static.kitster_banai' in all five quest rows (missing 't'); also add
  missing player_script and npc_script values to the smuggle quest row,
  which had blank script columns causing the quest to never activate.

- tekil_barje.tab: remove 26 trailing blank/junk lines ('16 - Extras'
  through '20 - Extras') that appeared after the three valid data rows;
  these could interfere with row-count parsing in the framework.

- blerx_tango.tab: clear stale deliver_object value from the first
  rescue quest row; rescue quests do not deliver objects and the
  mission_datadisk.iff path here was leftover copy-paste data.
@AconiteX
Copy link
Copy Markdown
Member

this one needs testing before merge

@nicolassanchez02
Copy link
Copy Markdown
Contributor Author

Spinning up a local SWG test server, will follow up with test results.

@nicolassanchez02
Copy link
Copy Markdown
Contributor Author

nicolassanchez02 commented May 19, 2026

Pulled this onto the test server and ran through what I could before going in-game.

Confirmed the bugs exist on master at the file level: the objection/tangible typo in rakir_banai rows 5 and 7, the static.kister_banai misspell across all 5 rows in kitster_banai plus the blank player_script/npc_script on the smuggle row, the 26 extra trailing lines in tekil_barje, and the deliver_object on the Mixon Guncher row in blerx_tango.

ant compile_tab on this branch is clean, all four .iff files generate without warnings. Dropped the .iff files in and the server comes up fine, nothing in the startup log touching those files.

Still need to go in-game and walk through the actual quest start/completion flow and verify the static.kitster_banai objvar tracks right. Will follow up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants