Skip to:

Opened 11 years ago

Closed 10 years ago

#625 closed defect (fixed)

Plugin activation/ wponce troubles

Reported by: so1o Owned by: mdawaffe
Milestone: 0.8.2 Priority: normal
Severity: normal Version: 0.7.5
Component: Back-end Keywords: has-patch


the plugin page activation/deactivation does not work if the plugin is in a subdirectory...

Attachments (2)

Ticket-625.2.patch (962 bytes) - added by so1o 11 years ago.
patch for plugin and theme fix
Ticket-625.patch (1.4 KB) - added by so1o 11 years ago.
patch for plugin and theme fix

Download all attachments as: .zip

Change History (30)

#1 @mdawaffe
11 years ago

  • Owner set to mdawaffe
  • Status changed from new to assigned

Works fine for me.

Is it just one subdirectory deep in the hierarchy?

Are you running this on Windows? I thought the new code was meant to avoid all Unix v. Windows slash issues, but... :)

#2 @so1o
11 years ago

its just one sub dir..


. what it does is asks for confirmation recurssively.

im running it on windows.. also one more thing.. everytime its adds more slashes to the dir so the first time it asks for confirmation the link shows


the next time its


#3 @mdawaffe
11 years ago

(In [789]) Fix? for plugin activation on windows. re #625

#4 @mdawaffe
11 years ago

so1o, can you try the above fix and see if it solves the problem?

#5 @so1o
11 years ago

no .. but interestingly this works...

$plugin = stripslashes(stripslashes(trim($_GET['plugin'])));

#6 @so1o
11 years ago

infact i just noticed im having this issue with the themes page also.. i think its the same slashes issue.

#7 @so1o
11 years ago

ok.. so here's the deal..

in many places we do this -

bb_nonce_url( add_query_arg( ..

this is causing the problem.

add_query_arg function converts \ to
. and internally bb_nonce_url calls add_query_arg so at the end of it we have four \s for each \.

michael.. thoughts?

#8 @mdawaffe
11 years ago

Are you running the WordPress script at the same time? If so, what version?

#9 @so1o
11 years ago

the installation is not integrated.

i had put in some debugging comments in the code.. it runs the add_query_arg from the wp_functions file.

just a sidenot we should change the calls to add_query_arg to bb_add_query_arg

#10 @so1o
11 years ago

duh.. please ignore my last 'sidenote'.. i was off my medication when i wrote the 'sidenote'

#11 @mdawaffe
11 years ago

so1o, what do the following functions return?


#12 @mdawaffe
11 years ago

Also, what do you get if you


#13 @so1o
11 years ago







array(3) {
  string(13) "anonymous.php"
  string(37) "forum-moderators\forum-moderators.php"
  string(9) "roles.php"

#14 @mdawaffe
11 years ago

(In [802]) add_query_arg() to [WP5193] re #625

#15 @mdawaffe
11 years ago

so1o, can you try that latest update to see if it works?

If it does, does it still work if you take out [789] (the stripslashes)?

#16 @so1o
11 years ago

actually it doesnt.. i'll try a couple of stunts of my own here..

#17 @so1o
11 years ago


dangit! it was a dud!

here is the patch!

#18 @so1o
11 years ago

  • Keywords has-patch added

11 years ago

patch for plugin and theme fix

#19 @so1o
11 years ago

had the same problem in themes.. updated the patch

#20 @so1o
11 years ago

ok the final patch for this.. i promise!

11 years ago

patch for plugin and theme fix

#21 @mdawaffe
11 years ago

I think this is all a result of a couple inconsistencies in add_query_arg()

#WP4084 and #WP4105

I'm going to wait until the dust settles on them.

#22 @so1o
11 years ago

sure thing.

#23 @so1o
11 years ago

this also messes up the pagination

the url comes out as forum.php?id=1&page=%25_%25 and is forwarded to the paginate_links

which will not work.

#24 @mdawaffe
11 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [808]) add_query_arg() and urlencode_deep() from WP. Fixes #625

#25 @mdawaffe
11 years ago


Can you verify this works on Windows too?

#26 @so1o
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Nope not working! investigation underway...

#27 @mdawaffe
10 years ago

  • Milestone changed from 1.0 to 0.8.2

#28 @mdawaffe
10 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

Think it was the same thing as #558

Marking as fixed.

Note: See TracTickets for help on using tickets.