Craig Hawker
Craig Hawker
Sounds like a great first PR if anyone is interested.
Definitely going to apply the "bug" label to this, as I can't see the reason this would be like this unless it were by accident. That said: early next year...
Note: this issue only happens if the `Schedule` class is used directly. Using a `Frequency` instead is fine, as that already has a JsonConverter in place to control the deserialization.
There should be a number of unit tests added around this to check that any changes do not adversely affect the overall deserialization process.
When the [task queue is registered](https://github.com/M-Files/VAF.Extensions.Community/blob/master/MFiles.VAF.Extensions/TaskQueueBackgroundOperations/TaskQueueBackgroundOperationManager/TaskQueueBackgroundOperationManager.cs#L176) it does not provide any details on the queue type (sequential or concurrent), nor does it provide any explicit value for the maximum concurrency....
The current internal ExpandSimpleConcatenation method uses ToPropertyIds to retrieve an array of property IDs for each match. ToPropertyIds takes a single match and returns the property ID "path" to iterate...
The current (internal) method [`ToPropertyIds`](https://github.com/M-Files/VAF.Extensions.Community/blob/master/MFiles.VAF.Extensions/ExtensionMethods/ObjVerEx/ExpandSimpleConcatenation.cs#L177) does exactly this, although it operates on a regex group rather than a simple string. Potentially this could be exposed. I don't think that it...
That sounds potentially reasonable. Can you give me some more information on why exactly you're trying to find these list items, though? I'm interested to see whether we should architect...
The approach that the current code takes (by using extension methods to generate everything) was quick to implement, but isn't very flexible or extensible, and I don't like the idea...
Please submit it as a PR. I really like the concept. I had a very quick look through the repo and the only thing that caught my eye was ensuring...