Skip to content

Commit 5acc32b

Browse files
Remove XML parsers from core-api #10650 (#11429)
* Remove XmlPartDescriptorParser #11434 * Remove XmlPageDescriptorParser #11435 * Remove XmlLayoutDescriptorParser #11436 * Remove XmlMixinDescriptorParser #11433 * Remove XmlDataDescriptorParser #11432 * Remove XmlStyleDescriptorParser #11431 * Remove XmlSiteDescriptorParser #11438 Split site.xml into two descriptors #11250 * Remove XmlContentTypeDescriptorParser #11437 * Remove XmlMacroDescriptorParser #11446 * Remove XmlApplicationParser #11447 * Remove XmlServiceDescriptorParser #11451 * Remove XmlTaskDescriptorParser #11452 * Remove XmlIdProviderDescriptorParser #11455 * Remove XmlWidgetDescriptorParser #11454 * Remove XmlAdminToolDescriptorParser #11449 * Remove XmlApiDescriptorParser #11450 * Remove XmlWebappDescriptorParser #11453 * Move XmlNodeParser and cleanup #11460 * Move XmlNodeParser and cleanup #11460
1 parent 6b99f95 commit 5acc32b

File tree

384 files changed

+2736
-8163
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

384 files changed

+2736
-8163
lines changed

modules/admin/admin-api/src/main/java/com/enonic/xp/admin/tool/AdminToolDescriptor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ public DescriptorKeys getApiMounts()
108108

109109
public static ResourceKey toResourceKey( final DescriptorKey key )
110110
{
111-
return ResourceKey.from( key.getApplicationKey(), "admin/tools/" + key.getName() + "/" + key.getName() + ".xml" );
111+
return ResourceKey.from( key.getApplicationKey(), "admin/tools/" + key.getName() + "/" + key.getName() + ".yml" );
112112
}
113113

114114
public static ResourceKey toIconResourceKey( final DescriptorKey key )

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/tool/AdminToolDescriptorBuilderMapper.java

Lines changed: 0 additions & 33 deletions
This file was deleted.

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/tool/AdminToolDescriptorServiceImpl.java

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
import com.enonic.xp.resource.ResourceProcessor;
1616
import com.enonic.xp.resource.ResourceService;
1717
import com.enonic.xp.security.PrincipalKeys;
18-
import com.enonic.xp.xml.XmlException;
1918

2019
@Component(immediate = true)
2120
public final class AdminToolDescriptorServiceImpl
@@ -87,24 +86,6 @@ private ResourceProcessor<DescriptorKey, AdminToolDescriptor> createProcessor( f
8786

8887
private AdminToolDescriptor loadDescriptor( final DescriptorKey key, final Resource resource )
8988
{
90-
final AdminToolDescriptor.Builder builder = AdminToolDescriptor.create().key( key );
91-
parseXml( resource, builder );
92-
return builder.build();
93-
}
94-
95-
private void parseXml( final Resource resource, final AdminToolDescriptor.Builder builder )
96-
{
97-
try
98-
{
99-
final XmlAdminToolDescriptorParser parser = new XmlAdminToolDescriptorParser();
100-
parser.currentApplication( resource.getKey().getApplicationKey() );
101-
parser.builder( builder );
102-
parser.source( resource.readString() );
103-
parser.parse();
104-
}
105-
catch ( final Exception e )
106-
{
107-
throw new XmlException( e, "Could not load admin app descriptor [" + resource.getKey() + "]: " + e.getMessage() );
108-
}
89+
return YmlAdminToolDescriptorParser.parse( resource.readString(), key.getApplicationKey() ).key( key ).build();
10990
}
11091
}

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/tool/XmlAdminToolDescriptorParser.java

Lines changed: 0 additions & 66 deletions
This file was deleted.

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/tool/YmlAdminToolDescriptorParser.java

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,13 @@
11
package com.enonic.xp.admin.impl.tool;
22

3+
import com.fasterxml.jackson.annotation.JsonProperty;
4+
35
import com.enonic.xp.admin.tool.AdminToolDescriptor;
46
import com.enonic.xp.app.ApplicationKey;
57
import com.enonic.xp.core.impl.schema.YmlParserBase;
8+
import com.enonic.xp.descriptor.DescriptorKeys;
9+
import com.enonic.xp.schema.LocalizedText;
10+
import com.enonic.xp.security.PrincipalKeys;
611

712
public final class YmlAdminToolDescriptorParser
813
{
@@ -17,4 +22,22 @@ public static AdminToolDescriptor.Builder parse( final String resource, final Ap
1722
{
1823
return PARSER.parse( resource, AdminToolDescriptor.Builder.class, currentApplication );
1924
}
25+
26+
private abstract static class AdminToolDescriptorBuilderMapper
27+
{
28+
@JsonProperty("displayName")
29+
public abstract AdminToolDescriptor.Builder displayName( LocalizedText text );
30+
31+
@JsonProperty("description")
32+
public abstract AdminToolDescriptor.Builder description( LocalizedText text );
33+
34+
@JsonProperty("allow")
35+
public abstract AdminToolDescriptor.Builder addAllowedPrincipals( PrincipalKeys allowedPrincipals );
36+
37+
@JsonProperty("apis")
38+
public abstract AdminToolDescriptor.Builder apiMounts( DescriptorKeys apiDescriptors );
39+
40+
@JsonProperty("interfaces")
41+
public abstract AdminToolDescriptor.Builder addInterfaces( Iterable<String> interfaceNames );
42+
}
2043
}

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/widget/WidgetDescriptorBuilderMapper.java

Lines changed: 0 additions & 32 deletions
This file was deleted.

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/widget/WidgetDescriptorLoader.java

Lines changed: 6 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@
88

99
import com.enonic.xp.admin.widget.WidgetDescriptor;
1010
import com.enonic.xp.app.ApplicationKey;
11+
import com.enonic.xp.descriptor.DescriptorKey;
1112
import com.enonic.xp.descriptor.DescriptorKeyLocator;
1213
import com.enonic.xp.descriptor.DescriptorKeys;
1314
import com.enonic.xp.descriptor.DescriptorLoader;
1415
import com.enonic.xp.icon.Icon;
15-
import com.enonic.xp.descriptor.DescriptorKey;
1616
import com.enonic.xp.resource.Resource;
1717
import com.enonic.xp.resource.ResourceKey;
1818
import com.enonic.xp.resource.ResourceService;
@@ -49,20 +49,16 @@ public DescriptorKeys find( final ApplicationKey key )
4949
@Override
5050
public ResourceKey toResource( final DescriptorKey key )
5151
{
52-
return ResourceKey.from( key.getApplicationKey(), PATH + "/" + key.getName() + "/" + key.getName() + ".xml" );
52+
return ResourceKey.from( key.getApplicationKey(), PATH + "/" + key.getName() + "/" + key.getName() + ".yml" );
5353
}
5454

5555
@Override
5656
public WidgetDescriptor load( final DescriptorKey key, final Resource resource )
5757
{
58-
final WidgetDescriptor.Builder builder = WidgetDescriptor.create();
59-
builder.key( key );
60-
61-
final String descriptorXml = resource.readString();
62-
parseXml( key.getApplicationKey(), builder, descriptorXml );
63-
final Icon icon = loadIcon( key );
64-
builder.setIcon( icon );
65-
return builder.build();
58+
return YmlWidgetDescriptorParser.parse( resource.readString(), key.getApplicationKey() )
59+
.key( key )
60+
.setIcon( loadIcon( key ) )
61+
.build();
6662
}
6763

6864
@Override
@@ -77,15 +73,6 @@ public WidgetDescriptor postProcess( final WidgetDescriptor descriptor )
7773
return descriptor;
7874
}
7975

80-
private void parseXml( final ApplicationKey applicationKey, final WidgetDescriptor.Builder builder, final String xml )
81-
{
82-
final XmlWidgetDescriptorParser parser = new XmlWidgetDescriptorParser();
83-
parser.builder( builder );
84-
parser.currentApplication( applicationKey );
85-
parser.source( xml );
86-
parser.parse();
87-
}
88-
8976
private Icon loadIcon( final DescriptorKey key )
9077
{
9178
final String iconPath = PATH + "/" + key.getName() + "/" + key.getName() + ".svg";

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/widget/XmlWidgetDescriptorParser.java

Lines changed: 0 additions & 66 deletions
This file was deleted.

modules/admin/admin-impl/src/main/java/com/enonic/xp/admin/impl/widget/YmlWidgetDescriptorParser.java

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,12 @@
11
package com.enonic.xp.admin.impl.widget;
22

3+
import com.fasterxml.jackson.annotation.JsonProperty;
4+
35
import com.enonic.xp.admin.widget.WidgetDescriptor;
46
import com.enonic.xp.app.ApplicationKey;
57
import com.enonic.xp.core.impl.schema.YmlParserBase;
8+
import com.enonic.xp.schema.LocalizedText;
9+
import com.enonic.xp.security.PrincipalKeys;
610

711
public final class YmlWidgetDescriptorParser
812
{
@@ -17,4 +21,22 @@ public static WidgetDescriptor.Builder parse( final String resource, final Appli
1721
{
1822
return PARSER.parse( resource, WidgetDescriptor.Builder.class, currentApplication );
1923
}
24+
25+
private abstract static class WidgetDescriptorBuilderMapper
26+
{
27+
@JsonProperty("displayName")
28+
public abstract WidgetDescriptor.Builder displayName( LocalizedText text );
29+
30+
@JsonProperty("description")
31+
public abstract WidgetDescriptor.Builder description( LocalizedText text );
32+
33+
@JsonProperty("allow")
34+
public abstract WidgetDescriptor.Builder allowedPrincipals( PrincipalKeys allowedPrincipals );
35+
36+
@JsonProperty("interfaces")
37+
public abstract WidgetDescriptor.Builder addInterfaces( Iterable<String> interfaceNames );
38+
39+
@JsonProperty("config")
40+
public abstract WidgetDescriptor.Builder addProperty( String key, String value );
41+
}
2042
}

0 commit comments

Comments
 (0)