diff --git a/docs/upgrades/packages-7.7.0/root_import_storyarchive_storyarchive.wgpkg b/docs/upgrades/packages-7.7.0/root_import_storyarchive_storyarchive.wgpkg new file mode 100644 index 000000000..386ad0b8d Binary files /dev/null and b/docs/upgrades/packages-7.7.0/root_import_storyarchive_storyarchive.wgpkg differ diff --git a/docs/upgrades/packages-7.7.0/root_import_storymanager_editstory.wgpkg b/docs/upgrades/packages-7.7.0/root_import_storymanager_editstory.wgpkg index 96fc86fd3..2e087d9b9 100644 Binary files a/docs/upgrades/packages-7.7.0/root_import_storymanager_editstory.wgpkg and b/docs/upgrades/packages-7.7.0/root_import_storymanager_editstory.wgpkg differ diff --git a/lib/WebGUI/Asset/Wobject/StoryArchive.pm b/lib/WebGUI/Asset/Wobject/StoryArchive.pm index ed7f49628..983ae46a0 100644 --- a/lib/WebGUI/Asset/Wobject/StoryArchive.pm +++ b/lib/WebGUI/Asset/Wobject/StoryArchive.pm @@ -108,16 +108,14 @@ sub definition { fieldType => 'template', label => $i18n->get('template'), hoverHelp => $i18n->get('template help'), - filter => 'fixId', namespace => 'StoryArchive', - defaultValue => '', + defaultValue => 'yxD5ka7XHebPLD-LXBwJqw', }, storyTemplateId => { tab => 'display', fieldType => 'template', label => $i18n->get('story template'), hoverHelp => $i18n->get('story template help'), - filter => 'fixId', namespace => 'Story', defaultValue => '', }, @@ -126,7 +124,6 @@ sub definition { fieldType => 'template', label => $i18n->get('edit story template'), hoverHelp => $i18n->get('edit story template help'), - filter => 'fixId', namespace => 'Story/Edit', defaultValue => 'E3tzZjzhmYoNlAyP2VW33Q', }, @@ -207,7 +204,8 @@ sub getFolder { isHidden => 1, }); $newVersionTag->commit(); - $oldVersionTag->setWorking(); + ##Restore the old one, if it exists + $oldVersionTag->setWorking() if $oldVersionTag; ##Get a new version of the asset from the db with the correct state $folder = WebGUI::Asset->newByUrl($session, $folderUrl); @@ -266,6 +264,8 @@ sub view { #This is an example of debugging code to help you diagnose problems. #WebGUI::ErrorHandler::warn($self->get("templateId")); + use Data::Dumper; + $session->log->warn(Dumper $var); return $self->processTemplate($var, undef, $self->{_viewTemplate}); } @@ -306,16 +306,19 @@ sub viewTemplateVariables { if ($storyDate ne $lastStoryDate) { push @{ $var->{date_loop} }, {}; $datePointer = $var->{date_loop}->[-1]; - $datePointer->{epochDate} = $creationDate; - $datePointer->{storyLoop} = []; + $datePointer->{epochDate} = $creationDay; + $datePointer->{story_loop} = []; $lastStoryDate = $storyDate; } - push @{$datePointer->{storyLoop}}, { + push @{$datePointer->{story_loop}}, { url => $story->getUrl, title => $story->getTitle, creationDate => $creationDate } } + + $var->{addStoryUrl} = $self->getUrl('func=add;class=WebGUI::Asset::Story'); + $var->{canPostStories} = $self->canPostStories; return 1; } diff --git a/lib/WebGUI/i18n/English/Asset_StoryArchive.pm b/lib/WebGUI/i18n/English/Asset_StoryArchive.pm index e25680a37..88959c04d 100644 --- a/lib/WebGUI/i18n/English/Asset_StoryArchive.pm +++ b/lib/WebGUI/i18n/English/Asset_StoryArchive.pm @@ -165,6 +165,12 @@ our $I18N = { lastUpdated => 0, }, + 'add a story' => { + message => q|Add a Story.|, + context => q|label for the URL to add a story to the archive.|, + lastUpdated => 0, + }, + }; 1; diff --git a/t/Asset/Wobject/StoryArchive.t b/t/Asset/Wobject/StoryArchive.t index 7e62b4748..55db09389 100644 --- a/t/Asset/Wobject/StoryArchive.t +++ b/t/Asset/Wobject/StoryArchive.t @@ -70,12 +70,15 @@ my $class = 'WebGUI::Asset::Wobject::StoryArchive'; my $loaded = use_ok($class); my $storage; +my $versionTag; SKIP: { skip "Unable to load module $class", $tests unless $loaded; -$archive = WebGUI::Asset->getDefault($session)->addChild({className => $class}); +$archive = WebGUI::Asset->getDefault($session)->addChild({className => $class, title => 'My Stories', url => '/home/mystories'}); +$versionTag = WebGUI::VersionTag->getWorking($session); +$versionTag->commit; isa_ok($archive, 'WebGUI::Asset::Wobject::StoryArchive', 'created StoryArchive'); @@ -155,9 +158,15 @@ is($folder->getFirstChild->getTitle, 'First Story', '... and it is the correct c # ################################################################ -my $oldFolder = $archive->getFolder(997966800); -my $tomorrow = time()+24*3600; +my $wgBday = 997966800; +my $oldFolder = $archive->getFolder($wgBday); + +my $tomorrow = time()+24*3600; my $newFolder = $archive->getFolder($tomorrow); + +my ($wgBdayMorn,undef) = $session->datetime->dayStartEnd($wgBday); +my ($tomorrowMorn,undef) = $session->datetime->dayStartEnd($tomorrow); + my $story = $oldFolder->addChild({ className => 'WebGUI::Asset::Story', title => 'WebGUI is released'}); $session->db->write('update asset set creationDate=997966800 where assetId=?',[$story->getId]); $story = $newFolder->addChild({ className => 'WebGUI::Asset::Story', title => "There's always tomorrow" }); @@ -174,26 +183,28 @@ KEY: foreach my $key (keys %{ $templateVars }) { cmp_deeply( $templateVars, { - date_loop => [ + canPostStories => 0, + addStoryUrl => '/home/mystories?func=add;class=WebGUI::Asset::Story', + date_loop => [ { - epochDate => 997966800, - storyLoop => [ { - creationDate => 997966800, - url => '/home/untitled/august_16_2001/webgui-is-released', + epochDate => $wgBdayMorn, + story_loop => [ { + creationDate => $wgBday, + url => '/home/mystories/august_16_2001/webgui-is-released', title => 'WebGUI is released', }, ], }, { epochDate => ignore(), - storyLoop => [ { + story_loop => [ { creationDate => ignore(), url => re('first-story'), title => 'First Story', }, ], }, { - epochDate => $tomorrow, - storyLoop => [{ + epochDate => $tomorrowMorn, + story_loop => [{ creationDate => $tomorrow, url => re('theres-always-tomorrow'), title => "There's always tomorrow", @@ -209,6 +220,7 @@ $folder->addChild({ className => 'WebGUI::Asset::Story', title => 'Story 3'}); $folder->addChild({ className => 'WebGUI::Asset::Story', title => 'Story 4'}); $archive->update({storiesPerPage => 3}); +$session->user({userId => 3}); $templateVars = {}; $archive->viewTemplateVariables($templateVars); @@ -217,21 +229,24 @@ KEY: foreach my $key (keys %{ $templateVars }) { delete $templateVars->{$key}; } + cmp_deeply( $templateVars, { - date_loop => [ + canPostStories => 1, + addStoryUrl => '/home/mystories?func=add;class=WebGUI::Asset::Story', + date_loop => [ { - epochDate => 997966800, - storyLoop => [ { - creationDate => 997966800, - url => '/home/untitled/august_16_2001/webgui-is-released', + epochDate => $wgBdayMorn, + story_loop => [ { + creationDate => $wgBday, + url => '/home/mystories/august_16_2001/webgui-is-released', title => 'WebGUI is released', }, ], }, { epochDate => ignore(), - storyLoop => [ + story_loop => [ { creationDate => ignore(), url => re('first-story'), @@ -258,7 +273,9 @@ END { if (defined $archive and ref $archive eq $class) { $archive->purge; } - WebGUI::VersionTag->getWorking($session)->rollback; + if ($versionTag) { + $versionTag->rollback; + } foreach my $user ($editor, $reporter, $reader) { $user->delete if defined $user; }