Skip to:

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#672 closed defect (bug) (wontfix)

Check if the content is false in save() of Wire class + diff

Reported by: nicolagreco's profile nicolagreco Owned by:
Milestone: Priority: major
Severity: Version: 1.0
Component: Keywords: has-patch, nicolagreco


Assuming that a spam filter mark a wire content as spam, so it changes $wire->content = false via the newest filter bp_wire_post_content ([1327])

the content could be false

and if the content will be false it has not to be saved in the db

look at the diff file

Attachments (1)

bp-wire-content-check.diff (414 bytes) - added by nicolagreco 15 years ago.

Download all attachments as: .zip

Change History (4)

#1 @DJPaul
15 years ago

Problem is this removes original content not allowing for manual checking/validation etc. There are always false positives.

#2 @apeatling
15 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

This is not a good idea, I've added before and after save actions in r1351 that can be used for this.

#3 @(none)
15 years ago

  • Milestone Wire 1.0 deleted

Milestone Wire 1.0 deleted

Note: See TracTickets for help on using tickets.