Jump to content
Strawberry Orange Banana Lime Leaf Slate Sky Blueberry Grape Watermelon Chocolate Marble
Strawberry Orange Banana Lime Leaf Slate Sky Blueberry Grape Watermelon Chocolate Marble

MSFN is made available via donations, subscriptions and advertising revenue. The use of ad-blocking software hurts the site. Please disable ad-blocking software or set an exception for MSFN. Alternatively, register and become a site sponsor/subscriber and ads will be disabled automatically. 



Francesco

[BUG+FIX] Saving after command drag&drop breaks the commands list

Recommended Posts

Francesco    0

Cause

This is caused by two different issues:

  1. The function that populates the command textbox searches the array trough Id even though when drag&dropping the Id stop reflecting the current grid position.
  2. When you drag&drop an item the commands array is not updated so fixing the function for populating the textbox is not enough, the array has to be refreshed after each drag&drop as well.

Fix

In configwizard.js replace

CommandsGrid.attachEvent("onRowSelect",FillInCommand);

with


CommandsGrid.attachEvent("onRowSelect",function (rowId) {
FillInCommand(CommandsGrid.getRowIndex(rowId));
});
CommandsGrid.attachEvent("onDrop", ExtractCommandsValues);

Please report if this change breaks anything.

Edited by Francesco

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now


  • Recently Browsing   0 members

    No registered users viewing this page.

×