PPL

Clicking to edit the Username from the Search page does not save

Bug #718428 reported by Christopher Adams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PPL
Fix Released
Critical
Bassel Safadi

Bug Description

Viewing the profile page from the dashboard or from the profile, the Owner can click to edit a Username and Save it.

If the User is viewing the profile loaded on the search page, then clicking save does not work.

To reproduce this:
   Use the Search Bar in the header and enter the name of one of your contacts.
   You will be redirected to the [root]/search page where you will see a People List with your contact.
   Click your contacts name; the profile will load on the right side of the search page.
   Click the username and change it. You will see the name change.
   Reload the search page or check the db. The full_name of that record has not been changed.

Again there must be some JS driving this that is not present on the search page.

Changed in ppl:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → Brad Phillips (brad-bradphillips)
milestone: none → 0.2
Revision history for this message
Bassel Safadi (bassel) wrote :

ok this is a bit tricky:
here is why this is happening:
the edit in place markup of <edit></edit> is also the edit in place handler which reads the input and insert it. so in the case of changing names in the dashboard, the post goes to /dashboard where the edit in place exists.

in the case of search and since the widget 45 doesn't have the url search in it's path. when one do edit in place the post goes to /search but on /search there is no <edit></edit> since the edit is coming from widget 45 which was called via ajax call.
for this to work the post of edit in place inside /search must go to /dashboard
so I added a new feature to aiki edit in place
you can now specify to which url the post goes by adding the following in aiki <edit></edit> markup, in this case:

<post_to>[root]/dashboard</post_to>

I added this to the edit in place on widget 45 so it force the request to always go to /dashboard even if it was on search..

Changed in ppl:
status: Confirmed → Fix Released
assignee: Brad Phillips (brad-bradphillips) → Bassel Safadi (bassel)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.