← Back to team overview

mudlet-makers team mailing list archive

Re: [Bug 1377401] Re: Script content loss

 

Would I be right in thinking that this thing "like Topsy" has got too
big (and complex) to track everything that is going on.

You might like to check, but I think all the OptionsArea stuff is just
useless cruft (all dlgOptionsArea*.h; mpOptionsArea* pointers in
dlgTRiggerEditor and all the options_area_*.ui forms) this is not doing
anything to harm or cause the problems but they do not seem to be adding
anything to the project in this area AFAICS.

I vaguely recall AC suggesting that he wanted to go over the trigger
editor - I take it that he is not currently able to add any insight into
this matter.

I'm guessing but I'm beginning to suspect that each of the
{Action|Alias|Keys|Scripts|Triggers}MainArea classes (and possibly the
Vars one as well) will have to gain an isDirty flag which is set when
any content (after we have picked out what *is* content - does
"mpActionsMainArea->lineEdit_action_icon" count, for instance) is
modified and use that to determine whether to save the current
content... (that is compounded by the complexity of whether the current
content is a "group" or "item" {branch vs leaf node???})

I'm guessing we want something for the delta version if not before, 8-(

On 08/01/15 00:04, Vadim Peretokin wrote:
> Agreed - that's what I did. I ended up with a loop through that didn't seem
> fixable.

-- 
You received this bug notification because you are a member of Mudlet
Makers, which is subscribed to Mudlet.
https://bugs.launchpad.net/bugs/1377401

Title:
  Script content loss

Status in Mudlet the MUD client:
  Confirmed

Bug description:
  People are reporting that their script are getting wiped on 3.0.0-alpha[1] and beta[2]. 
  This is a regression in 3.0 as it doesn't happen in 2.1. Steps to replicate:

  - Select a script with your mouse
  - Navigate up or down at least 1 item with your arrow keys
  - Switch to any other tab
  - Switch back to the original script

  Result: original script is blank.

  See http://forums.mudlet.org/viewtopic.php?f=7&t=4610&p=21779#p21778
  for the original report.

  [1] http://forums.mudlet.org/viewtopic.php?f=5&t=4598&p=21563#p21563

  [2] http://forums.mudlet.org/viewtopic.php?f=7&t=4610&p=21775#p21774

To manage notifications about this bug go to:
https://bugs.launchpad.net/mudlet/+bug/1377401/+subscriptions


References