[wp-trac] Re: [WordPress Trac] #10181: Warnings after post/page save
WordPress Trac
wp-trac at lists.automattic.com
Mon Jun 15 23:36:25 GMT 2009
#10181: Warnings after post/page save
--------------------------+-------------------------------------------------
Reporter: Viper007Bond | Type: defect (bug)
Status: new | Priority: normal
Milestone: 2.8.1 | Component: Autosave
Version: 2.8 | Severity: blocker
Keywords: has-patch |
--------------------------+-------------------------------------------------
Comment(by Denis-de-Bernardy):
posting the IRC logs for reference:
{{{
1:31 AM ddebernardy
I think there are two issues
1:31 AM Viper007Bond
there are
1:31 AM ddebernardy
one is we don't check that it's writable
1:31 AM Viper007Bond
(it is)
1:31 AM ddebernardy
the is_writable check is different for IIS and apache, but we can manage
that
1:31 AM Viper007Bond
i'm on apache
1:31 AM
the file is 777
1:31 AM ddebernardy
the other is that we're trying to do this a dozen times, and that
definitely isn't normal
1:32 AM Viper007Bond
one write per line or something
1:32 AM
foreach ( $markerdata as $n => $markerline ) {
1:32 AM
oh, wait, maybe not
1:32 AM
hmm
1:33 AM ddebernardy
nah, I'd think it's related to do_action(save_post) et al being called
multiple times (once per post, and once per autosave, and once per
revision, etc, and possibly more than once each
1:33 AM Viper007Bond
save_mod_rewrite_rules() is getting called too often
1:34 AM
infact it doesn't even need to be called at all when you save a post
1:34 AM
not since like 2.0.x
1:34 AM ddebernardy
it used to, when the rules were all in the htaccess file
1:34 AM Viper007Bond
yep
1:34 AM ddebernardy
but indeed, now they don't, we merely need to update the option thingy
1:35 AM
even then, though, we need to make sure it only ever gets called once when
needed
}}}
--
Ticket URL: <http://core.trac.wordpress.org/ticket/10181#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list