Fix T47255: inset polygon addon left old edges

Not sure why this used to work. The old code used the bmesh faceseq
delete, which kills only the faces. This fix uses the utility
bmesh.ops.delete() with an arg that will maybe kill edges and verts too.
Also made code select all the new polys. This is not ideal (ideal would
be to select only the inner ones), but making the ideal change requires
more work.
This commit is contained in:
Howard Trickey
2016-02-01 10:15:59 -05:00
parent c3f24386bb
commit 9fe70215a0

View File

@ -151,10 +151,10 @@ def do_inset(mesh, amount, height, region, as_percent):
for i in range(orig_numv, len(m.points.pos)):
bvertnew = bm.verts.new(m.points.pos[i])
bm.verts.index_update()
bm.verts.ensure_lookup_table()
new_faces = []
start_faces = len(bm.faces)
for i, newf in enumerate(blender_faces):
bm.verts.ensure_lookup_table()
vs = remove_dups([bm.verts[j] for j in newf])
if len(vs) < 3:
continue
@ -167,13 +167,13 @@ def do_inset(mesh, amount, height, region, as_percent):
# bfacenew.copy_from_face_interp(oldface)
else:
bfacenew = bm.faces.new(vs)
# remove original faces
new_faces.append(bfacenew)
# deselect original faces
for face in selfaces:
face.select_set(False)
bm.faces.remove(face)
bm.faces.index_update()
# mesh.update(calc_edges=True)
# select all new faces
# remove original faces
bmesh.ops.delete(bm, geom=selfaces, context=5) # 5 = DEL_FACES
# select all new faces (should only select inner faces, but that needs more surgery on rest of code)
for face in new_faces:
face.select_set(True)