Skip Menu |

This queue is for tickets about the RT-Client-REST CPAN distribution.

Report information
The Basics
Id: 118729
Status: resolved
Priority: 0/
Queue: RT-Client-REST
People
Owner: dean [...] fragfest.com.au
Requestors: DROLSKY [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sat Nov 12 13:32:39 2016 DROLSKY [...] cpan.org - Ticket created
Subject: Content can be lost because of regex for "not set" in RT::Client::REST::Object
The problem bit of code is: if ($value =~ m/not set/i) { $value = undef; } This can easily cause content to be set as undef if it contains the string "not set".
  Sat Nov 12 13:50:08 2016 DROLSKY [...] cpan.org - Correspondence added
On 2016-11-12 12:32:39, DROLSKY wrote: Show quoted text
> The problem bit of code is: > > if ($value =~ m/not set/i) { > $value = undef; > } > > This can easily cause content to be set as undef if it contains the > string "not set".
It might be enough to simply check that $key ne 'content' here. I'm not sure what all the possible keys are.
  Sat May 05 09:50:31 2018 dean [...] fragfest.com.au - Correspondence added
"Not set" is only returned for dates as far as i can tell from the rt code - just in RT/Date.pm
  Sat May 05 09:50:33 2018 The RT System itself - Status changed from 'new' to 'open'
  Sat May 05 09:50:33 2018 dean [...] fragfest.com.au - Taken
  Sat May 05 10:33:38 2018 dean [...] fragfest.com.au - Correspondence added
I have created this branch for changes i have developed https://github.com/RT-Client-REST/RT-Client-REST/tree/rt118729
  Thu Apr 30 18:31:15 2020 dean [...] fragfest.com.au - Correspondence added
fixed in 0.58
  Thu Apr 30 18:31:18 2020 dean [...] fragfest.com.au - Status changed from 'open' to 'resolved'