handle location being null
This commit is contained in:
parent
460ea7894d
commit
5bcc03f9d1
|
@ -10,6 +10,7 @@
|
|||
#### Fixed
|
||||
- Mark event-related data as non-translatable within plugin
|
||||
- Add version number to script registration to break browser caching
|
||||
- Handle location being null
|
||||
#### Security
|
||||
|
||||
### [1.2.0]
|
||||
|
|
|
@ -26,7 +26,7 @@ final class Formatter
|
|||
return $dateText;
|
||||
}
|
||||
|
||||
public static function format_location(string $description, string $locality): string {
|
||||
public static function format_location(string $description, ?string $locality): string {
|
||||
$location = '';
|
||||
if ($description && trim($description)) {
|
||||
$location .= trim($description);
|
||||
|
|
|
@ -42,6 +42,10 @@ final class FormatterTest extends PHPUnit\Framework\TestCase
|
|||
$this->assertSame('a', Formatter::format_location('a', ''));
|
||||
}
|
||||
|
||||
public function testLocationFormatDescriptionOnlyWithNull(): void {
|
||||
$this->assertSame('a', Formatter::format_location('a', null));
|
||||
}
|
||||
|
||||
public function testLocationFormatDescriptionWithSpaceOnly(): void {
|
||||
$this->assertSame('', Formatter::format_location(' ', ''));
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue