GM has no way of setting data for NPCs

Bug #60348 reported by Jonathan Lange
2
Affects Status Importance Assigned to Milestone
Joybot
Fix Committed
Critical
Unassigned

Bug Description

This includes adding and removing defenses, and setting DEX and SPD.

I suggest the following interface:
   <gm> npc foo set dex=14
   <gm> npc foo don chainmail

The implementation should probably be something like:
    def cmd_npc(self, params):
        name, rest = params.split(None, 1)
        c = self.getCharacter(name)
        c.execute(params)

This obviously wouldn't work (Character doesn't have execute), but you get the gist.

Revision history for this message
Chris Halse Rogers (raof) wrote :

Could be implemented by faking an Avatar, too, I think.
Critical for actual GM-use.
It also exposes the lack of differentiation between GM & players, though. We'll need to add some command-restriction stuff soon.

Changed in joybot:
importance: Untriaged → Critical
status: Unconfirmed → Confirmed
Revision history for this message
Chris Halse Rogers (raof) wrote :

Preliminary fix committed to joybot-npc, but needs cleaning:
*- Metamind could, perhaps, be a powerup on the Store
*- Split out into a separate plugin

Changed in joybot:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.