nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

Bug in Modifying Feature?

SusanFarley

I have two Node layers in my map with the second layer being a copy of the first. I am enabling users to merge similar nodes to help clean up data, but all alterations are done on the copy layer in case the user later decides those nodes shouldn't be merged. So in my code I have:

//MergeRow is the new row and SelectedNodes is the list of rows to be deleted

public async Task<bool> MergeNodes(DataRow MergedRow, List<DataRow> SelectedNodes) {

var modifyFeature = new EditOperation();

modifyFeature.Name = "Merged Nodes";

//Other stuff

modifyFeature.Delete(MergeNodeLayer, Int64.Parse(SelectedNodes[0][oidNum].ToString()));

modifyFeature.Execute();

return true;

}

Even though I pass in the MergeNodeLayer, it deletes the node from both the control layer and the new layer. Is this a bug with delete or am I doing something wrong? The nodes on both layers have the same oids since it is a direct copy so it is deleting the correct one from each layer, but it should only be deleting the one for the layer I passed in.

Thank you,

Susan

Encontre mais publicações marcadas com

Accepted answers

Anonymous User

Hi Susan,

To get the underlying featureclass path from a feature layer you'll have to concatenate the datastore path and the featureclass name. You can then feed this into the "mva" then into the geoprocessing tool. Note that if the featureclass was in a featuredataset in the datastore you would include that in the path too (determined from featureclass.GetFeatureDataset()).

Here's an example

      QueuedTask<SPAN class="punctuation token">.</SPAN><SPAN class="token function">Run</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="keyword token">async</SPAN> <SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN> <SPAN class="operator token">=</SPAN><SPAN class="operator token">></SPAN>
      <SPAN class="punctuation token">{</SPAN>
        <SPAN class="comment token">//find hydrant layer</SPAN>
        <SPAN class="keyword token">var</SPAN> hydLayer <SPAN class="operator token">=</SPAN> MapView<SPAN class="punctuation token">.</SPAN>Active<SPAN class="punctuation token">.</SPAN>Map<SPAN class="punctuation token">.</SPAN><SPAN class="token function">FindLayers</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="string token">"Hydrant"</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">.</SPAN><SPAN class="token function">FirstOrDefault</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN> <SPAN class="keyword token">as</SPAN> FeatureLayer<SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//get the underlying featureclass path for the layer</SPAN>
        <SPAN class="keyword token">var</SPAN> hydClass <SPAN class="operator token">=</SPAN> hydLayer<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetFeatureClass</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//get the datastore for the featureclass</SPAN>
        <SPAN class="keyword token">var</SPAN> hydStore <SPAN class="operator token">=</SPAN> hydClass<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetDatastore</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//concat hydrant path (datastore and featureclass name)</SPAN>
        <SPAN class="keyword token">var</SPAN> hydPath <SPAN class="operator token">=</SPAN> hydStore<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetPath</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">.</SPAN>AbsolutePath <SPAN class="operator token">+</SPAN> <SPAN class="string token">@"/"</SPAN> <SPAN class="operator token">+</SPAN> hydClass<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetName</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//make the value array and pass to the GP tool</SPAN>
        <SPAN class="keyword token">var</SPAN> mva <SPAN class="operator token">=</SPAN> Geoprocessing<SPAN class="punctuation token">.</SPAN><SPAN class="token function">MakeValueArray</SPAN><SPAN class="punctuation token">(</SPAN>@hydPath<SPAN class="punctuation token">,</SPAN> $<SPAN class="string token">@"{hydPath}2"</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>
        <SPAN class="keyword token">await</SPAN> Geoprocessing<SPAN class="punctuation token">.</SPAN><SPAN class="token function">ExecuteToolAsync</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="string token">"Copy_management"</SPAN><SPAN class="punctuation token">,</SPAN> mva<SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>
      <SPAN class="punctuation token">}</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN><SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

All comments

SusanFarley

Worked great! Thank you! Not sure if it helps you, but our customer number is below 1000. I can find out the exact number if you need it.

Anonymous User

Hi Susan,

Here's an example

      QueuedTask<SPAN class="punctuation token">.</SPAN><SPAN class="token function">Run</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="keyword token">async</SPAN> <SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN> <SPAN class="operator token">=</SPAN><SPAN class="operator token">></SPAN>
      <SPAN class="punctuation token">{</SPAN>
        <SPAN class="comment token">//find hydrant layer</SPAN>
        <SPAN class="keyword token">var</SPAN> hydLayer <SPAN class="operator token">=</SPAN> MapView<SPAN class="punctuation token">.</SPAN>Active<SPAN class="punctuation token">.</SPAN>Map<SPAN class="punctuation token">.</SPAN><SPAN class="token function">FindLayers</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="string token">"Hydrant"</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">.</SPAN><SPAN class="token function">FirstOrDefault</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN> <SPAN class="keyword token">as</SPAN> FeatureLayer<SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//get the underlying featureclass path for the layer</SPAN>
        <SPAN class="keyword token">var</SPAN> hydClass <SPAN class="operator token">=</SPAN> hydLayer<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetFeatureClass</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//get the datastore for the featureclass</SPAN>
        <SPAN class="keyword token">var</SPAN> hydStore <SPAN class="operator token">=</SPAN> hydClass<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetDatastore</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//concat hydrant path (datastore and featureclass name)</SPAN>
        <SPAN class="keyword token">var</SPAN> hydPath <SPAN class="operator token">=</SPAN> hydStore<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetPath</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">.</SPAN>AbsolutePath <SPAN class="operator token">+</SPAN> <SPAN class="string token">@"/"</SPAN> <SPAN class="operator token">+</SPAN> hydClass<SPAN class="punctuation token">.</SPAN><SPAN class="token function">GetName</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>

        <SPAN class="comment token">//make the value array and pass to the GP tool</SPAN>
        <SPAN class="keyword token">var</SPAN> mva <SPAN class="operator token">=</SPAN> Geoprocessing<SPAN class="punctuation token">.</SPAN><SPAN class="token function">MakeValueArray</SPAN><SPAN class="punctuation token">(</SPAN>@hydPath<SPAN class="punctuation token">,</SPAN> $<SPAN class="string token">@"{hydPath}2"</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>
        <SPAN class="keyword token">await</SPAN> Geoprocessing<SPAN class="punctuation token">.</SPAN><SPAN class="token function">ExecuteToolAsync</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="string token">"Copy_management"</SPAN><SPAN class="punctuation token">,</SPAN> mva<SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>
      <SPAN class="punctuation token">}</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN><SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

SusanFarley

I'm not sure if this is correct, but I have been trying this which is also not working.

foreach (BasicFeatureLayer fmLayer in lyr) //there is a node layer and a link layer

{

string newLayerName = String.Format("{0}_NewCopy", frmLayer.Name);

string pathToFeature = $@{gdbPath}\{newLayerName};

string pathFromFeature = $@{gdbPath}\{frmLayer.Name}";

await QueuedTask.Run(() =>

{

using (Table tbl = fmLayer.GetTable())

{

var mva = Geoprocessing.MakeValueArray(tbl, pathToFeature );

var res = await Geoprocessing.ExecuteToolAsync("management.Copy", mva);

}

res.Exception is null, res.Status = WaitingToRun, and res.Result = null.

I noticed that mva contains only strings when it is sent to Copy_management. Do you know how I correct this?

Thank you,

Susan

SusanFarley

The problem is that I was trying to Copy the FeatureLayer and that doesn't work with Copy. Do you know how I can get the Feature Dataset from the Feature Layer?

Thank you,

Susan

Anonymous User

Hmm, it looks reasonable. The syntax for the GP tool name ("management.Copy" or ""Copy_managment") will work either way so thats not the issue.

Make sure the pathFromFeature and pathToFeature strings make sense for a featureclass and you could also try those manually in the geoprocessing copy tool to see if it works there.

Your using the layer name as the feature class input which may not always be the same, particularly if the layer name has a space, the corresponding featureclass name wont.

Lastly, check the res variable (an IGPResult) to see the error message.

SusanFarley

I changed my code to (I'm guessing C# translation of your code above):

foreach (BasicFeatureLayer fmLayer in lyr) //there is a node layer and a link layer

{

string newLayerName = String.Format("{0}_NewCopy", frmLayer.Name);

string pathToFeature = $@{gdbPath}\{newLayerName};

string pathFromFeature = $@{gdbPath}\{frmLayer.Name}";

var mva = Geoprocessing.MakeValueArray(pathFromFeature , pathToFeature );

var res = await Geoprocessing.ExecuteToolAsync("management.Copy", mva);

}

And it does nothing so I'm guessing I am doing something wrong. Any ideas what?

Thank you,

Susan

SusanFarley

I'm guessing the C# version of that is:

foreach (BasicFeatureLayer fmLayer in lyr) //there is a node layer and a link layer

{

string newLayerName = String.Format("{0}_NewCopy", frmLayer.Name);

string pathToFeature = $@{gdbPath}\{newLayerName};

List<object> args = new List<object>() { gdbPath, newLayerName, featType.ToString(), "",

hasMeasure, "DISABLED"};

var res = await Geoprocessing.ExecuteToolAsync("management.CreateFeatureClass",

Geoprocessing.MakeValueArray(args.ToArray()));

await QueuedTask.Run(() => {

var mva = Geoprocessing.MakeValueArray(fmLayer, pathToFeature );

Geoprocessing.ExecuteToolAsync("management.CopyFeatures", mva);

}

Two questions:

Is there a faster way to accomplish this? For two small layers (only 8270 nodes), this takes 1 minute and 24 seconds. The 1:24 is for copying both layers.
On the new copies of the layers, it always seems to remove the first column of the attribute table (be it FID or Object_ID), replace it with Object_Id, but restart the numbering so that it is different than the original layer. Is there a way to prevent this? The user is looking or nodes and links in the "original layers" to merge in the "copied layers" so this causes an issue since

var modifyInspector = new Inspector(true);

modifyInspector.Load( MergeNodeLayer, refId); // this looks at the first column data and, with it being

//renumbered, will not edit the same node selected from the first layer as in the

//merge layer.

Example:

Original Layer

FID Shape Name Type

0 .... Street PolyLine

1 .... Road PolyLine

Copied Layer

OBJECTID Shape Name Type

1 .... Street PolyLine

2 .... Road PolyLine

Anonymous User

Susan,

The easiest way is to use the geoprocessing copy features tool.

      <SPAN class="keyword token">var</SPAN> mva <SPAN class="operator token">=</SPAN> Geoprocessing<SPAN class="punctuation token">.</SPAN><SPAN class="token function">MakeValueArray</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="string token">@"Hydrant"</SPAN><SPAN class="punctuation token">,</SPAN> <SPAN class="string token">@"D:\arcgis\ArcTutor\Editing\Zion.gdb\hydrant2"</SPAN><SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN>
      Geoprocessing<SPAN class="punctuation token">.</SPAN><SPAN class="token function">ExecuteToolAsync</SPAN><SPAN class="punctuation token">(</SPAN><SPAN class="string token">"CopyFeatures_management"</SPAN><SPAN class="punctuation token">,</SPAN> mva<SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">;</SPAN><SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

This will copy selected features from the input layer (or all if none are selected) into the new feature class and add a new layer to the current map. You wont need to copy the layer with the LayerFactory.

SusanFarley

await QueuedTask.Run(() => {

if (LayerFactory.Instance.CanCopyLayer(frmLayer)) {

var newLayer = LayerFactor.Instance.CopyLayer(fmLayer, MapView.Active.Map);

newLayer.SetName (String.Format("{0}_Copy", frmLayer.Name)

}

How do I make it create a copy of the featureclass as well?

Thank you,

Susan

Anonymous User

Susan,

It sounds like both layers in the map may be pointing to the same feature class. You can check this under the layer properties source tab for each.

How did you copy the layer?

Participe da Conversa

Membros conectados podem postar, seguir atualizações e mais. Novo aqui? Registre uma conta gratuita.

Links Rápidos

Community Help
Find useful guides, FAQs, and documents to help you navigate and make the most of Esri Community.