Project

General

Profile

Bug #5712

System Patches package: system_patches_edit.php - Patch contents is cleared when error happens

Added by Renato Botelho over 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Category:
System Patches
Target version:
Start date:
12/29/2015
Due date:
% Done:

0%

Estimated time:
Affected Version:
2.3
Affected Architecture:

Description

Add a new patch, then fill 'patch contents' with a patch and 'URL/Commit ID' with invalid data then press 'Save'.

It'll report error and remove content from 'patch contents'

History

#1 Updated by Steve Beaver over 3 years ago

  • Subject changed from System Patches: Patch contents is cleared when error happens to System Patches package: system_patches_edit.php - Patch contents is cleared when error happens
  • Status changed from New to Assigned
  • Assignee changed from Steve Beaver to Jim Pingle

It would appear that this is due to the patch being base64 encoded when saved to the config (no error) but being plain text when the page reloads after an error.

This is a simple way to fix it:

system_patches_edit.php line 184:

$patchtext = new Form_Textarea(
    'patch',
    'Patch Contents',
    $input_errors ? $pconfig['patch'] : base64_decode($pconfig['patch'])
);

#2 Updated by Jim Pingle over 3 years ago

  • Status changed from Assigned to Feedback

Fix committed

#3 Updated by Steve Beaver over 3 years ago

  • Assignee changed from Jim Pingle to Renato Botelho

#4 Updated by Steve Beaver over 3 years ago

  • Status changed from Feedback to Resolved

#5 Updated by Chris Buechler about 3 years ago

  • Affected Version set to 2.3

Also available in: Atom PDF