Prefab brush throw an error about 2d-extras HOT 9 CLOSED

@bqback I solved that problem by Erasing before painting. Something like this before Undo.MoveGameObjectToScene:

Erase(grid, brushTarget, position);

from 2d-extras.

Unity 2018.3.0f2

i try to modify ‘PrefabBrush.cs’, it seems solved, but can not limited to children of the currently selected GameObject.

public class PrefabBrush : GridBrushBase
=>
public class PrefabBrush : GridBrush

then

public class PrefabBrushEditor : GridBrushEditorBase
=>
public class PrefabBrushEditor : GridBrushEditor

then

protected void OnEnable()
{
m_SerializedObject = new SerializedObject(target);
=>
protected override void OnEnable()
{base.OnEnable();
m_SerializedObject = new SerializedObject(target);

from 2d-extras.

reproduced in 2018.3.0f2

from 2d-extras.

reproduced in 2018.3.0f1

from 2d-extras.

Hi, this will be fixed in a future version of Unity (both for 2018 and 2019 lines)!

from 2d-extras.

The error happens because after placing the object, the active tilemap is set to "Nothing" and so you get a null reference exception. Until there is a real fix, you can get around it by resetting the tilemap (click away from it and then click back on it) every time you place an object.

from 2d-extras.

Unity 2018.3.0f2

i try to modify ‘PrefabBrush.cs’, it seems solved, but can not limited to children of the currently selected GameObject.

public class PrefabBrush : GridBrushBase
=>
public class PrefabBrush : GridBrush

then

public class PrefabBrushEditor : GridBrushEditorBase
=>
public class PrefabBrushEditor : GridBrushEditor

then

protected void OnEnable()
{
m_SerializedObject = new SerializedObject(target);
=>
protected override void OnEnable()
{base.OnEnable();
m_SerializedObject = new SerializedObject(target);

This solved it for me! Thanks!

from 2d-extras.

Unity 2018.3.0f2

i try to modify ‘PrefabBrush.cs’, it seems solved, but can not limited to children of the currently selected GameObject.

public class PrefabBrush : GridBrushBase
=>
public class PrefabBrush : GridBrush

then

public class PrefabBrushEditor : GridBrushEditorBase
=>
public class PrefabBrushEditor : GridBrushEditor

then

protected void OnEnable()
{
m_SerializedObject = new SerializedObject(target);
=>
protected override void OnEnable()
{base.OnEnable();
m_SerializedObject = new SerializedObject(target);

Thank you for this fix (worked on 2018.3.0f2)! Although there's a problem: while the brush no longer throws an error and resets, it now draws a new sprite for every little movement of my mouse cursor within a single tile (so rapidly moving my mouse back and forth across a few pixels creates dozens of the same object), which I doubt to be the intended effect, Same goes for the eraser -- moving the eraser back and forth within a single tile allows me to quickly erase all of those excess sprites.

from 2d-extras.

This has been fixed in Unity 2018.3.4f1, or you can continue to use the PrefabBrush as modified!

from 2d-extras.