Warning Improvements by Xon 2.11.6

[Anthony Parsons]

@Mr Lucky, the fix is above your post. Run that query to fix the issue.

 

[Mr Lucky]

Doh, thanks!

 

[Mr Lucky]

Ran the query, is this to be expected:

0 rows affected. (Query took 0.0002 seconds.)

 

[Xon]

Xon updated Warning Improvements by Xon with a new update entry:

2.1.6 - Bugfix update

• Fix for "Trying to get property 'Category' of non-object" which could crop up.
  

Read the rest of this update entry...

 

[Xon]

Xon updated Warning Improvements by Xon with a new update entry:

2.1.7 - Maintenance update

• Fix "Please enter valid title " error when using the Warning Service from custom code with a custom warning definition.

Read the rest of this update entry...

 

[Mr Lucky]

Some phrases seem to be not rendered (2.1.7)

 

[Xon]

@Mr Lucky Can you verify if those phrases have any contents via the AdminCP => phrases section?

If they do, can you add a space to the text and click "save" then remove it and click save again?

 

[Mr Lucky]

Neither of those phrases titles turn up in a phrase search

 

[Xon]

Neither of those phrases titles turn up in a phrase search

I'm baffled at how those where lost then. If you edit the warning and the category and give them usable names again it should be fixed

 

[Mr Lucky]

warning_title.11 is fixed when I edit the warning as you suggest.

But I don't have any categories, the one at the top (sv_warning_category_title.1) is not there when I disable the addon.

 

[Mr Lucky]

Would it be possible to have View own warnings permission based? Thanks.

 

[Xon]

But I don't have any categories, the one at the top (sv_warning_category_title.1) is not there when I disable the addon.

You'll need to edit the warning category, update the title and then save it.

Would it be possible to have View own warnings permission based? Thanks.

I'm currently focusing on porting my paid add-ons, so this would be a low priority.

 

[Mr Lucky]

You'll need to edit the warning category, update the title and then save it.

But I don't have any categories

 

[Xon]

But I don't have any categories

The add-on automatically creates one called "Warning" when you install the add-on, and then assigns any warnings to it. This url path should let you use it; /admin.php?warnings/category-edit&warning_category_id=1

If the row is really gone, you'll need to manually re-add it by running this SQL;

INSERT INTO xf_sv_warning_category (warning_category_id, parent_category_id, display_order, allowed_user_group_ids)
VALUES (1, NULL, 0, 2)

 

[Mr Lucky]

Ah, OK I didn't know it would create a category called warnings. I've edited the phrase and called it warnings all fine now thanks.

 

[Xon]

Xon updated Warning Improvements by Xon with a new update entry:

2.1.8 - Maintenance update

• Support the "All user groups" option for Warning category permissions
  • This changes the defaults so this add-on no longer requires configuration for warnings to keep working on initial install
• Migrate Warning Category user group allowed list to known-good format
• Do not allow original Warning Category id 1 to be deleted

Read the rest of this update entry...

 

[Mr Lucky]

It seems that if I have options set so users cannot view their own warnings, if they click on an alert they get an error page.

Obviously the alert in this case cannot link to a warning, but in that case it would better if users got a more useful message other than Ooops We ran into some problems.

Alternatively it could work if users can be prevented from viewing their warnings in their profile, but only be able to view their own warnings when it is an alert.

 

[Xon]

Obviously the alert in this case cannot link to a warning, but in that case it would better if users got a more useful message other than Ooops We ran into some problems.

It should give a permission denied error, not an "oops we ran into problems". I'll look into that this weekend.

I'll probably have it remove the link if there isn't any permission.

 

[Mr Lucky]

It should give a permission denied error, not an "oops we ran into problems". I'll look into that this weekend.

I'll probably have it remove the link if there isn't any permission.

Yes, no link at all would be fine

 

[ŽivaAkcija]

any chance to add option fake news and hate speach?