never call get_user_data directly

This commit is contained in:
Daniel Friesel 2019-03-18 18:47:52 +01:00
parent 842fcbcea9
commit 07b3ea19a6
3 changed files with 5 additions and 2 deletions

View file

@ -677,6 +677,9 @@ helper 'get_user_token' => sub {
return;
};
# This helper should only be called directly when also providing a user ID.
# If you don't have one, use current_user() instead (get_user_data will
# delegate to it anyways).
helper 'get_user_data' => sub {
my ( $self, $uid ) = @_;

View file

@ -1,5 +1,5 @@
<h1>Account</h1>
% my $acc = get_user_data();
% my $acc = current_user();
<div class="row">
<div class="col s12">
<table class="striped">

View file

@ -34,7 +34,7 @@
<div class="container">
% if (is_user_authenticated()) {
% my $acc = get_user_data();
% my $acc = current_user();
% if ($acc and $acc->{deletion_requested}) {
%= include '_deletion_note', timestamp => $acc->{deletion_requested}
% }