viewField() doesn't consider that getCacheability() can be NULL

Issue created by @joachim
First commit to issue fork.
Merge request !13Update file ComputedFieldBuilder.php → (Open) created by kalpanajaiswal
Pipeline finished with Success
6 months ago
Total: 171s
#203073
Comment 6 months ago →
🇮🇳India kalpanajaiswal
Raised the MR with condition.
Status changed to Needs review 6 months ago5:23pm 19 June 2024
Comment 6 months ago →
🇮🇳India kalpanajaiswal
Comment 6 months ago →
🇮🇳India jagraj_singh_gill
According to @joachim 's suggestion instead of checking the instance I have created the patch to check the truthy value. Please review once. Thanks !
Merge request !14Condition applied to check the truthy value of getCacheability → (Open) created by jagraj_singh_gill
Pipeline finished with Success
6 months ago
Total: 140s
#203813
Comment 6 months ago →
🇮🇳India jagraj_singh_gill
According to @joachim 's suggestion instead of checking the instance I have applied the condition check the truthy value. I have placed the MR ,please review once. Thanks !
Pipeline finished with Success
6 months ago
#203814

joachim → committed ca2488ee on 4.0.x

Issue #3455750 by kalpanajaiswal, jagraj_singh_gill: Fixed viewField()...

Status changed to Fixed 6 months ago8:50am 24 June 2024
Comment 6 months ago →
🇬🇧United Kingdom joachim
Committed. Thanks!
Comment 5 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Merge Requests