Bug 22408

Hi Owen,
This is the mail I've received form the sandbox : 

The sandbox you've requested is not ready.
Some problems occurred applying patches from bug 22408:
<h1>Something went wrong !</h1>Applying: Bug 22408: Subscription entry form cleanup
fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt).
Patch failed at 0001 Bug 22408: Subscription entry form cleanup
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem run git bz apply --continue.
If you would prefer to skip this patch, instead run git bz apply --skip.
To restore the original branch and stop patching run git bz apply --abort.
error: could not build fake ancestor
Bug 22408 - Subscription entry form cleanup

85630 - Bug 22408: Subscription entry form cleanup

Apply? [(y)es, (n)o, (i)nteractive] Patch left in /tmp/Bug-22408-Subscription-entry-form-cleanup-wLHUqi.patch .

I rebased Bug 22250, which I think was causing the problem. Thanks for testing!

Applying: Bug 22408: Subscription entry form cleanup
error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt).
error: could not build fake ancestor
Patch failed at 0001 Bug 22408: Subscription entry form cleanup

Setting to Patch doesn't apply

Created attachment 88189 [details] [review]
Bug 22408: Subscription entry form cleanup

This patch makes a number of changes to the subscription entry form:

  - Make the default layout single-column.
  - Switch the layout to two-column only when testing prediction
    pattern.
    - Add a button to hide the prediction pattern test pane.
  - Change the style of some buttons and links. Add Font Awesome icons
    to some.
  - Clean up some issues with capitalization and spacing.
  - Make it clearer when form fields are read-only: Move "locked" form
    field style from addbiblio.css to staff-global.css and improve it
    with regard to mouse interactions

To test, apply the patch and regenerate the staff client CSS. Clear your
browser cache if necessary.

 - Go to Serials -> New subscription.
 - Readonly fields under "Vendor" and "Record" should show a small
   padlock icon.
 - Test the process of adding a subscription, paying special attention
   to the advanced pattern controls: Showing, hiding, modifying,
   canceling.
 - Test the process of editing a subscription in the same way.

Failed QA

Steps taken
- Set Record for New Subscription and clicked Next, clicking Okay to continue without a Vendor.
- Set Issue publication date, set Frequency 1 week, did not specify subscription length
- Set subscription start date and end date.
- Created an advanced pattern and named it, then created another advanced pattern and chose it as the Numbering pattern above
- Clicked test prediction pattern and selected two dates to not publish randomly
- Clicked Save subscription
- Error!

Error trace:

Can't call method "title" on an undefined value at /home/vagrant/kohaclone/Koha/Subscription.pm line 165

1.    in Koha::Subscription::get_sharable_info at /home/vagrant/kohaclone/Koha/Subscription.pm line 165

      162:     my $sub_numberpatteern =
      163:       Koha::Subscription::Numberpatterns->find( $subscription->numberpattern );
      164: 
      165:     my $sub_mana_info = {
      166:         'title'           => $biblio->title,
      167:         'sfdescription'   => $sub_frequency->description,
      168:         'unit'            => $sub_frequency->unit,

2.   in Koha::SharedContent::prepare_entity_data at /home/vagrant/kohaclone/Koha/SharedContent.pm line 123

      120:     my $ressource_mana_info;
      121:     my $packages = "Koha::".ucfirst($ressourcetype)."s";
      122:     my $package = "Koha::".ucfirst($ressourcetype);
      123:     $ressource_mana_info = $package->get_sharable_info($ressourceid);
      124:     $ressource_mana_info = { %$ressource_mana_info, %$mana_info };
      125: 
      126:     return $ressource_mana_info;

3.    in Koha::SharedContent::send_entity at /home/vagrant/kohaclone/Koha/SharedContent.pm line 82

       79:     my ($lang, $loggedinuser, $resourceid, $resourcetype, $content) = @_;
       80: 
       81:     unless ( $content ) {
       82:         $content = prepare_entity_data($lang, $loggedinuser, $resourceid, $resourcetype);
       83:     }
       84: 
       85:     my $result = process_request(build_request('post', $resourcetype, $content));

Sorry, I couldn't reproduce this problem. Do you have Mana turned on and configured? Did you test the same process in master?

Thank you for testing!

Created attachment 88953 [details] [review]
Bug 22408: Subscription entry form cleanup

This patch makes a number of changes to the subscription entry form:

  - Make the default layout single-column.
  - Switch the layout to two-column only when testing prediction
    pattern.
    - Add a button to hide the prediction pattern test pane.
  - Change the style of some buttons and links. Add Font Awesome icons
    to some.
  - Clean up some issues with capitalization and spacing.
  - Make it clearer when form fields are read-only: Move "locked" form
    field style from addbiblio.css to staff-global.css and improve it
    with regard to mouse interactions

To test, apply the patch and regenerate the staff client CSS. Clear your
browser cache if necessary.

 - Go to Serials -> New subscription.
 - Readonly fields under "Vendor" and "Record" should show a small
   padlock icon.
 - Test the process of adding a subscription, paying special attention
   to the advanced pattern controls: Showing, hiding, modifying,
   canceling.
 - Test the process of editing a subscription in the same way.

Signed-off-by: Michal Denar <black23@gmail.com>

Created attachment 88984 [details] [review]
Bug 22408: Subscription entry form cleanup

This patch makes a number of changes to the subscription entry form:

  - Make the default layout single-column.
  - Switch the layout to two-column only when testing prediction
    pattern.
    - Add a button to hide the prediction pattern test pane.
  - Change the style of some buttons and links. Add Font Awesome icons
    to some.
  - Clean up some issues with capitalization and spacing.
  - Make it clearer when form fields are read-only: Move "locked" form
    field style from addbiblio.css to staff-global.css and improve it
    with regard to mouse interactions

To test, apply the patch and regenerate the staff client CSS. Clear your
browser cache if necessary.

 - Go to Serials -> New subscription.
 - Readonly fields under "Vendor" and "Record" should show a small
   padlock icon.
 - Test the process of adding a subscription, paying special attention
   to the advanced pattern controls: Showing, hiding, modifying,
   canceling.
 - Test the process of editing a subscription in the same way.

Signed-off-by: Michal Denar <black23@gmail.com>

Awesome work all!

Pushed to master for 19.05

Enhancement will not be backported to 18.11.x series.