Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#304 closed bug (fixed)

MOD_layoutbits is breaking searchmembers ajax xml

Reported by: lemon-head Owned by: lemon-head
Priority: major Milestone: 0.1.2.1
Component: BW Search Keywords: searchmembers, MOD_layoutbits
Cc: tobixen

Description

The reason was: Two blanklines after "?>" in layoutbits.php.
This caused a linebreak before the <?xml statement in the returned xml file.

Change History (11)

comment:1 Changed 10 years ago by lemon-head

  • Owner set to lemon-head
  • Status changed from new to assigned

comment:2 Changed 10 years ago by lemon-head

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

should be fixed with [3707]

comment:3 Changed 10 years ago by philipp

  • follow_up changed from none to move to alpha
  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:4 Changed 10 years ago by matrixpoint

  • follow_up changed from move to alpha to test on alpha

Moved to alpha in r3712, r3713.

Doesn't work yet. When a search doesn't complete (the loading... message is never overwriting with the search result summary), one quick test is to open searchmembers/ajax, which gives the error:

Fatal error: Class 'MOD_layoutbits' not found in /home/bwrox/alpha.bewelcome.org/build/searchmembers/searchmembers.model.php on line 319

I suspect that another file is needed. The ajax page is not aware of the new class MOD_layoutbits. But I uploaded the new layoutbits module with r3712. It should be detected and loaded automatically.

comment:5 Changed 10 years ago by matrixpoint

I forgot to add the module.xml file. Uploaded in r3715. But now, searchmembers/ajax gives this error:

Fatal error: Call to undefined method MOD_words::getBuffered() in /home/bwrox/alpha.bewelcome.org/modules/layoutbits/lib/layoutbits.lib.php on line 103

So, there are still some word module files that need to be moved to alpha before this can be tested.

comment:6 Changed 10 years ago by micha

We can't test searchmembers on alpha anymore. The error is a long-known: The googlemaps-key is wrong!

comment:7 Changed 10 years ago by micha

ABQIAAAA-eKVrlUbv2I1NZ-U4kXTbBSquW6niBjr0AFMRa6hhXhRFZ-zJxRZouOKo2UHju3VDj8xT5XIwSLDLw old

ABQIAAAA-h6GWkeEHED3beQWCPHi5RTnd2erWePPER5A2i02q-ulKWabWxT0gJCZXyKsfknvCQ7IocFOyFLOgQ new, use this one!

comment:8 Changed 10 years ago by matrixpoint

  • Cc tobixen added
  • version changed from test to alpha

comment:9 Changed 10 years ago by lemon-head

  • Milestone set to 0.1.2.1 release with MOD_words + MOD_layoutbits

comment:10 Changed 10 years ago by micha

  • follow_up changed from test on alpha to release
  • Resolution set to fixed
  • Status changed from reopened to closed

Seems to work fine. Ready to move to production!

comment:11 Changed 10 years ago by matrixpoint

  • follow_up changed from release to none

Released in r3927, r3928.

Note: See TracTickets for help on using tickets.