Fix missing overrides in the Navigation when shortcutted.

Also, documented the getOverridesList method inside Shortcut.

docs/changelog/7.x.x.txt

@@ -7,6 +7,7 @@
  - fixed #10409: Problem using hash in metadata possible values
  - fixed a similar problem with metadata in the Shortcut Asset.
  - fixed #10396: Syndicated Content wobject not displaying edit controls
+ - fixed #10386: Template override missing in nav shortcut
 
 7.7.7
  - Added EMS Schedule table

lib/WebGUI/Asset/Shortcut.pm

@@ -368,6 +368,15 @@ sub getFieldsList {
 }
 
 #-------------------------------------------------------------------
+
+=head2 getOverridesList
+
+Generates a table of overrides, based on the shortcutted Asset's definition sub.
+If the field type is hidden, or the field label is empty, then an override is not
+generated for that field
+
+=cut
+
 sub getOverridesList {
 	my $self = shift;
 	my $output = '';

lib/WebGUI/Asset/Wobject/Navigation.pm

@@ -34,19 +34,21 @@ sub definition {
 		tableName=>'Navigation',
 		className=>'WebGUI::Asset::Wobject::Navigation',
 		properties=>{
-			templateId =>{
+            templateId => {
-				fieldType=>"template",
+                label        => $i18n->get(1096),
-				defaultValue=>'PBtmpl0000000000000048'
+                fieldType    => "template",
-				},
+                defaultValue => 'PBtmpl0000000000000048'
-			mimeType =>{
+            },
-				fieldType=>"mimeType",
+            mimeType => {
-				defaultValue=>'text/html'
+                label        => $i18n->get('mimeType'),
-				},
+                fieldType    => "mimeType",
-			assetsToInclude=>{
+                defaultValue => 'text/html'
-				fieldType=>'checkList',
+            },
-				defaultValue=>"descendants"
+            assetsToInclude => {
-				},
+                fieldType    =>'checkList',
-			startType=>{
+                defaultValue =>"descendants"
+            },
+			startType => {
 				fieldType=>'selectBox',
 				defaultValue=>"relativeToCurrentUrl"
 				},
@@ -62,22 +64,26 @@ sub definition {
 				fieldType=>'selectBox',
 				defaultValue=>55
 				},
-			showSystemPages=>{
+			showSystemPages => {
-				fieldType=>'yesNo',
+                label        => $i18n->get(30),
-				defaultValue=>0
+				fieldType    => 'yesNo',
-				},
+				defaultValue => 0,
-			showHiddenPages=>{
+            },
-				fieldType=>'yesNo',
+			showHiddenPages => {
-				defaultValue=>0
+                label        => $i18n->get(31),
-				},
+				fieldType    => 'yesNo',
-			showUnprivilegedPages=>{
+				defaultValue => 0,
-				fieldType=>'yesNo',
+            },
-				defaultValue=>0
+			showUnprivilegedPages => {
-				},
+                label        => $i18n->get(32),
-			reversePageLoop=>{
+				fieldType    => 'yesNo',
-				fieldType=>'yesNo',
+				defaultValue => 0,
-				defaultValue=>0
+            },
-				},
+			reversePageLoop => {
+                label        => $i18n->get('reverse page loop'),
+				fieldType    => 'yesNo',
+				defaultValue => 0,
+            },
 			}
 		});
         return $class->SUPER::definition($session, $definition);