Fixed a bug in the ...Entries tag where entries wouldn't be identif…

…ied depending upon which configuration options were used.
openmelody · Sep 15, 2014 · 5d47d42 · 5d47d42
1 parent c02e6e6
commit 5d47d42
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/addons/ConfigAssistant.pack/config.yaml b/addons/ConfigAssistant.pack/config.yaml
@@ -4,7 +4,7 @@ key: ConfigAssistant
 author_link: https://github.com/openmelody/mt-plugin-configassistant
 author_name: "Byrne Reese, Open Melody Software Group"
 description: This plugin provides a simple YAML based framework for creating plugin and theme configuration options.
-version: 2.5.12
+version: 2.5.13
 static_version: 20
 schema_version: 4
 

diff --git a/addons/ConfigAssistant.pack/lib/ConfigAssistant/Plugin.pm b/addons/ConfigAssistant.pack/lib/ConfigAssistant/Plugin.pm
@@ -427,10 +427,10 @@ sub _hdlr_field_entry_loop {
     my $value  = _get_field_value($ctx);
 
     # The value contains both active and inactive entries. We want the
-    # active ones, because the inactive ones aren't supposed to get 
+    # active ones, because the inactive ones aren't supposed to get
     # published. The format is, for example: `active:1,2,5;inactive:3,4,6`
-    ( my $active_ids = $value ) =~ s{active:(.*);inactive.*}{$1};
-    my @ids = grep { /^\d+$/ } split( ',', $active_ids );
+    ( my $active_ids = $value ) =~ s{active:([\d,]*)(;)?(inactive.*)?}{$1};
+    my @ids = grep { /\d+/ } split( ',', $active_ids );
 
     # There are no entries selected.
     unless ( @ids ) {