Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
30 Jul 2016

Summary of Changes

This removes the frontend file components/com_patchtester/patchtester.php it contains nothing and is not longer included into the package: https://github.com/joomla-extensions/patchtester/blob/master/build/patchtester/build.sh

Testing Instructions

This can't be tested.

avatar zero-24 zero-24 - open - 30 Jul 2016
avatar mbabker
mbabker - comment - 30 Jul 2016

Does it still install correctly without that file?

On Saturday, July 30, 2016, zero-24 notifications@github.com wrote:

Summary of Changes

This removes the frontend file components/com_patchtester/patchtester.php
it contains nothing and is not longer included into the package:
https://github.com/joomla-extensions/patchtester/blob/master/build/patchtester/build.sh
Testing Instructions

This can't be tested.

You can view, comment on, or merge this pull request online at:

#169
Commit Summary

  • remove fronted file
  • use the new cool script.php ;)

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#169, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAWfoffoSiazswjOa2gqfn14qIUl48AKks5qa7RLgaJpZM4JY4yZ
.

avatar zero-24
zero-24 - comment - 30 Jul 2016

Why not? It is not included in the build script. But i can double check if i'm back on my pc tomorrow :)

avatar mbabker
mbabker - comment - 30 Jul 2016

Please do. Joomla's a bit funny about components I've learned.

On Saturday, July 30, 2016, zero-24 notifications@github.com wrote:

Why not? It is not included in the build script. But i can double check if
i'm back on my pc tomorrow :)


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#169 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWfoU2Qr3hghpTs2Pzsc5U7WV1T9Ksoks5qa8nDgaJpZM4JY4yZ
.

avatar zero-24
zero-24 - comment - 31 Jul 2016

The frontend file is not included into the final package:
image
... but the install works ;)

avatar mbabker mbabker - close - 31 Jul 2016
avatar mbabker mbabker - merge - 31 Jul 2016
avatar mbabker mbabker - reference | b48b187 - 31 Jul 16
avatar mbabker mbabker - merge - 31 Jul 2016
avatar mbabker mbabker - close - 31 Jul 2016
avatar mbabker mbabker - change - 31 Jul 2016
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-07-31 15:48:00
Closed_By mbabker
avatar mbabker
mbabker - comment - 31 Jul 2016

Now I remember what the oddity is. You can create an admin only component without a frontend but you can't create a frotend only component without an admin (seriously the component adapter throws an Exception if you don't have that tag specified).

avatar zero-24 zero-24 - head_ref_deleted - 31 Jul 2016
avatar zero-24
zero-24 - comment - 31 Jul 2016

Thanks

Add a Comment

Login with GitHub to post a comment