Notification level of Settings is not changed.

Bug #513019 reported by nayuki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Jisko
Fix Released
Low
Marcos Garcia

Bug Description

Apology Monkey vomits the thing not read easily because of the machine translation.

settings.php
It doesn't update because the IF sentence doesn't pass when the variable is 0.
if ($notification_level)

if (isset($notification_level)) {
or
if(preg_match("/^[0-9]+$/",$num)){

Revision history for this message
Marcos Garcia (marcosgdf) wrote :

What version are you talking about? 2.0 or 3.0?

Thanks!

Revision history for this message
nayuki (kaji) wrote :

> What version are you talking about? 2.0 or 3.0?

What version are you talking about? 2.0 or 3.0?

Thanks!

Revision history for this message
nayuki (kaji) wrote :

I'm sorry.

The version is 2.0.

My best regards.

Revision history for this message
Marcos Garcia (marcosgdf) wrote :

Going to check it tomorrow.

Sorry for the delay.

Regards, Marcos.

Changed in jisko:
milestone: none → 2.0
importance: Undecided → Low
status: New → Confirmed
assignee: nobody → Marcos Garcia (marcosgdf)
status: Confirmed → In Progress
Revision history for this message
Marcos Garcia (marcosgdf) wrote :

Fixed in last revision.

Thanks!

Changed in jisko:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.