AllSpiceMirrors/kicad
7
Fork 0
mirror of https://gitlab.com/kicad/code/kicad.git synced 2025-04-04 23:35:31 +00:00
Files Issues Releases Activity

More functional fix for IPC API item modifications

Browse Source 
Fixes https://gitlab.com/kicad/code/kicad/-/issues/20206
This commit is contained in:
Jon Evans 2025-03-28 20:27:25 -04:00
parent 3bac9344d2
commit dc9fa2e2ac
19 changed files with 101 additions and 1 deletions

2
include/board_item.h
View File

@ -88,6 +88,8 @@ public:
~BOARD_ITEM();
virtual void CopyFrom( const BOARD_ITEM* aOther );
void SetParentGroup( PCB_GROUP* aGroup ) { m_group = aGroup; }
PCB_GROUP* GetParentGroup() const { return m_group; }

2
pcbnew/api/api_handler_pcb.cpp
View File

@ -478,7 +478,7 @@ HANDLER_RESULT<ItemRequestStatus> API_HANDLER_PCB::handleCreateUpdateItemsIntern
{
BOARD_ITEM* boardItem = *optItem;
commit->Modify( boardItem );
boardItem->SwapItemData( item.get() );
boardItem->CopyFrom( item.get() );
boardItem->Serialize( newItem );
}

7
pcbnew/board_item.cpp
View File

@ -44,6 +44,13 @@ BOARD_ITEM::~BOARD_ITEM()
}
void BOARD_ITEM::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther, /* void */ );
*this = *aOther;
}
const BOARD* BOARD_ITEM::GetBoard() const
{
if( Type() == PCB_T )

7
pcbnew/footprint.cpp
View File

@ -948,6 +948,13 @@ FOOTPRINT& FOOTPRINT::operator=( const FOOTPRINT& aOther )
}
void FOOTPRINT::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_FOOTPRINT_T, /* void */ );
*this = *static_cast<const FOOTPRINT*>( aOther );
}
bool FOOTPRINT::IsConflicting() const
{
return HasFlag( COURTYARD_CONFLICT );

2
pcbnew/footprint.h
View File

@ -135,6 +135,8 @@ public:
FOOTPRINT& operator=( const FOOTPRINT& aOther );
FOOTPRINT& operator=( FOOTPRINT&& aOther );
void CopyFrom( const BOARD_ITEM* aOther ) override;
void Serialize( google::protobuf::Any &aContainer ) const override;
bool Deserialize( const google::protobuf::Any &aContainer ) override;

7
pcbnew/pad.cpp
View File

@ -141,6 +141,13 @@ PAD& PAD::operator=( const PAD &aOther )
}
void PAD::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_PAD_T, /* void */ );
*this = *static_cast<const PAD*>( aOther );
}
void PAD::Serialize( google::protobuf::Any &aContainer ) const
{
using namespace kiapi::board::types;

2
pcbnew/pad.h
View File

@ -60,6 +60,8 @@ public:
PAD( const PAD& aPad );
PAD& operator=( const PAD &aOther );
void CopyFrom( const BOARD_ITEM* aOther ) override;
void Serialize( google::protobuf::Any &aContainer ) const override;
bool Deserialize( const google::protobuf::Any &aContainer ) override;

7
pcbnew/pcb_reference_image.cpp
View File

@ -81,6 +81,13 @@ PCB_REFERENCE_IMAGE& PCB_REFERENCE_IMAGE::operator=( const BOARD_ITEM& aItem )
}
void PCB_REFERENCE_IMAGE::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_REFERENCE_IMAGE_T, /* void */ );
*this = *static_cast<const PCB_REFERENCE_IMAGE*>( aOther );
}
EDA_ITEM* PCB_REFERENCE_IMAGE::Clone() const
{
return new PCB_REFERENCE_IMAGE( *this );

2
pcbnew/pcb_reference_image.h
View File

@ -43,6 +43,8 @@ public:
~PCB_REFERENCE_IMAGE();
void CopyFrom( const BOARD_ITEM* aOther ) override;
PCB_REFERENCE_IMAGE& operator=( const BOARD_ITEM& aItem );
/**

7
pcbnew/pcb_shape.cpp
View File

@ -69,6 +69,13 @@ PCB_SHAPE::~PCB_SHAPE()
}
void PCB_SHAPE::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_SHAPE_T, /* void */ );
*this = *static_cast<const PCB_SHAPE*>( aOther );
}
void PCB_SHAPE::Serialize( google::protobuf::Any &aContainer ) const
{
using namespace kiapi::common;

2
pcbnew/pcb_shape.h
View File

@ -46,6 +46,8 @@ public:
~PCB_SHAPE() override;
void CopyFrom( const BOARD_ITEM* aOther ) override;
static bool ClassOf( const EDA_ITEM* aItem )
{
return aItem && PCB_SHAPE_T == aItem->Type();

7
pcbnew/pcb_text.cpp
View File

@ -81,6 +81,13 @@ PCB_TEXT::~PCB_TEXT()
}
void PCB_TEXT::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_TEXT_T, /* void */ );
*this = *static_cast<const PCB_TEXT*>( aOther );
}
void PCB_TEXT::Serialize( google::protobuf::Any &aContainer ) const
{
using namespace kiapi::common;

2
pcbnew/pcb_text.h
View File

@ -46,6 +46,8 @@ public:
~PCB_TEXT();
void CopyFrom( const BOARD_ITEM* aOther ) override;
static inline bool ClassOf( const EDA_ITEM* aItem )
{
return aItem && PCB_TEXT_T == aItem->Type();

8
pcbnew/pcb_textbox.cpp
View File

@ -61,6 +61,14 @@ PCB_TEXTBOX::~PCB_TEXTBOX()
{
}
void PCB_TEXTBOX::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_TEXTBOX_T, /* void */ );
*this = *static_cast<const PCB_TEXTBOX*>( aOther );
}
void PCB_TEXTBOX::Serialize( google::protobuf::Any &aContainer ) const
{
using namespace kiapi::common::types;

2
pcbnew/pcb_textbox.h
View File

@ -43,6 +43,8 @@ public:
~PCB_TEXTBOX();
void CopyFrom( const BOARD_ITEM* aOther ) override;
static inline bool ClassOf( const EDA_ITEM* aItem )
{
return aItem && PCB_TEXTBOX_T == aItem->Type();

21
pcbnew/pcb_track.cpp
View File

@ -71,6 +71,13 @@ EDA_ITEM* PCB_TRACK::Clone() const
}
void PCB_TRACK::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_TRACE_T, /* void */ );
*this = *static_cast<const PCB_TRACK*>( aOther );
}
PCB_ARC::PCB_ARC( BOARD_ITEM* aParent, const SHAPE_ARC* aArc ) :
PCB_TRACK( aParent, PCB_ARC_T )
{
@ -86,6 +93,13 @@ EDA_ITEM* PCB_ARC::Clone() const
}
void PCB_ARC::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_ARC_T, /* void */ );
*this = *static_cast<const PCB_ARC*>( aOther );
}
PCB_VIA::PCB_VIA( BOARD_ITEM* aParent ) :
PCB_TRACK( aParent, PCB_VIA_T ),
m_padStack( this )
@ -136,6 +150,13 @@ PCB_VIA& PCB_VIA::operator=( const PCB_VIA &aOther )
}
void PCB_VIA::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_VIA_T, /* void */ );
*this = *static_cast<const PCB_VIA*>( aOther );
}
EDA_ITEM* PCB_VIA::Clone() const
{
return new PCB_VIA( *this );

6
pcbnew/pcb_track.h
View File

@ -124,6 +124,8 @@ public:
// Do not create a copy constructor. The one generated by the compiler is adequate.
void CopyFrom( const BOARD_ITEM* aOther ) override;
void Move( const VECTOR2I& aMoveVector ) override
{
m_Start += aMoveVector;
@ -311,6 +313,8 @@ public:
PCB_ARC( BOARD_ITEM* aParent, const SHAPE_ARC* aArc );
void CopyFrom( const BOARD_ITEM* aOther ) override;
static inline bool ClassOf( const EDA_ITEM *aItem )
{
return aItem && PCB_ARC_T == aItem->Type();
@ -411,6 +415,8 @@ public:
PCB_VIA( const PCB_VIA& aOther );
PCB_VIA& operator=( const PCB_VIA &aOther );
void CopyFrom( const BOARD_ITEM* aOther ) override;
bool IsType( const std::vector<KICAD_T>& aScanTypes ) const override
{
if( BOARD_CONNECTED_ITEM::IsType( aScanTypes ) )

7
pcbnew/zone.cpp
View File

@ -113,6 +113,13 @@ ZONE& ZONE::operator=( const ZONE& aOther )
}
void ZONE::CopyFrom( const BOARD_ITEM* aOther )
{
wxCHECK( aOther && aOther->Type() == PCB_ZONE_T, /* void */ );
*this = *static_cast<const ZONE*>( aOther );
}
ZONE::~ZONE()
{
delete m_Poly;

2
pcbnew/zone.h
View File

@ -80,6 +80,8 @@ public:
~ZONE();
void CopyFrom( const BOARD_ITEM* aOther ) override;
static inline bool ClassOf( const EDA_ITEM* aItem )
{
return aItem && aItem->Type() == PCB_ZONE_T;