Announcement

  •  » Engine
  •  » [AddUsersNotes] new trigger needed on pwg.users.getList

#1 2014-03-11 16:15:33

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
13791

[AddUsersNotes] new trigger needed on pwg.users.getList

Hello,

I've been working on plugin AddUsersNotes. It is a major rewrite since page [Administration > Users > Manage] has been completely rewritten for Piwigo 2.6.

It works fine, but to avoid a complex coding, I need:

Code:

[pierrick@pierrick-desktop] ~/public_html/piwigo/dev/branches/2.6
$ svn diff
Index: include/ws_functions/pwg.users.php
===================================================================
--- include/ws_functions/pwg.users.php  (revision 27489)
+++ include/ws_functions/pwg.users.php  (working copy)
@@ -251,6 +251,8 @@
     }
   }
 
+  $users = trigger_change('render_ws_users_getList', $users);
+  
   return array(
     'paging' => new PwgNamedStruct(
       array(

What do you think of it? OK for 2.6.2?

Offline

 

#2 2014-03-11 16:37:13

rvelices
Former Piwigo Team
2005-12-29
1960

Re: [AddUsersNotes] new trigger needed on pwg.users.getList

Ok I would not call it render though

Offline

 

#3 2014-03-11 17:05:39

flop25
Piwigo Team
2006-07-06
7037

Re: [AddUsersNotes] new trigger needed on pwg.users.getList

ok

I will have to had some trigger_action missing in the admin part for my next plugin "Take A Tour"

ps: does that mean that the 2.6.2 will be released soon?


To get a better help : Politeness like Hello-A link-Your past actions precisely described
Check my extensions : more than 30 available
who I am and what I do : http://fr.gravatar.com/flop25
My gallery : an illustration of how to integrate Piwigo in your website

Offline

 

#4 2014-03-11 19:32:36

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
13791

Re: [AddUsersNotes] new trigger needed on pwg.users.getList

rvelices wrote:

Ok I would not call it render though

No problem, which name for this trigger then?

Offline

 

#5 2014-03-11 19:34:05

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
13791

Re: [AddUsersNotes] new trigger needed on pwg.users.getList

flop25 wrote:

I will have to had some trigger_action missing in the admin part for my next plugin "Take A Tour"

OK.

flop25 wrote:

ps: does that mean that the 2.6.2 will be released soon?

I don't know. Considering 2.6.1 was nearly 2 months ago, yes 2.6.2 should come fast now.

Offline

 

#6 2014-03-11 19:38:55

rvelices
Former Piwigo Team
2005-12-29
1960

Re: [AddUsersNotes] new trigger needed on pwg.users.getList

plg wrote:

rvelices wrote:

Ok I would not call it render though

No problem, which name for this trigger then?

Just ws_users_getList...

Offline

 

#7 2014-03-12 14:45:37

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
13791

Re: [AddUsersNotes] new trigger needed on pwg.users.getList

Offline

 
  •  » Engine
  •  » [AddUsersNotes] new trigger needed on pwg.users.getList

Board footer

Powered by FluxBB

github twitter newsletter Donate Piwigo.org © 2002-2024 · Contact