Issue Details (XML | Word | Printable)

Key: CORE-5257
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Vlad Khorsun
Reporter: Vlad Khorsun
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Firebird Core

Enable nested keys in plugin configuration

Created: 02/Jun/16 09:27 PM   Updated: 25/May/17 08:21 PM
Component/s: API / Client Library, Engine
Affects Version/s: 3.0.0, 3.0.1
Fix Version/s: 3.0.2, 4.0 Alpha 1

QA Status: Cannot be tested


 Description  « Hide
Config-parsing code already supports nested keys but this feature is explicitly disabled in code for more than one nesting level.
API also supports nested keys, see IConfigEntry::getSubConfig method.
It seems we could safely enable this feature (and there is users asking for it)

Example of nested keys (could be used in plugins.conf):

Config = MyPluginConfig {
  Key1 = Val1
  Key2 = Val2 {
    SubKey = Val
    Subkey2 = {
...
   }
  }
}

currently, such configuration is parsed as set of plain keys.

 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Vlad Khorsun made changes - 02/Jun/16 09:27 PM
Field Original Value New Value
Assignee Vlad Khorsun [ hvlad ]
Vlad Khorsun made changes - 02/Jun/16 10:08 PM
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 3.0.1 [ 10730 ]
Fix Version/s 4.0 Alpha 1 [ 10731 ]
Resolution Fixed [ 1 ]
Vlad Khorsun added a comment - 24/Oct/16 08:10 PM
Re-opened due to wrong parsing of nested keys

Vlad Khorsun made changes - 24/Oct/16 08:10 PM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Fix Version/s 3.0.2 [ 10785 ]
Vlad Khorsun added a comment - 24/Oct/16 08:21 PM - edited
Strongly speaking, wrong parsing of nested keys is a different issue, but enabling of not working code is a nonsense therefore i decided to not create new ticket.

Vlad Khorsun made changes - 24/Oct/16 08:21 PM
Fix Version/s 3.0.1 [ 10730 ]
Affects Version/s 3.0.1 [ 10730 ]
Vlad Khorsun made changes - 27/Oct/16 03:54 PM
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Pavel Zotov made changes - 25/May/17 08:21 PM
Status Resolved [ 5 ] Resolved [ 5 ]
QA Status No test Cannot be tested